Skip to content

feat: cors 설정#159

Merged
ose0221 merged 2 commits intomainfrom
dev
Mar 13, 2026
Merged

feat: cors 설정#159
ose0221 merged 2 commits intomainfrom
dev

Conversation

@ose0221
Copy link
Contributor

@ose0221 ose0221 commented Mar 13, 2026

🚀 PR 요약

목적이 무엇인가요? - 지우고 작성

✨ PR 상세 내용

어떤 부분이 어떻게 변경이 되었나요? - 지우고 작성

🚨 주의 사항

주의할 부분이 무엇인가요? - 지우고 작성

✅ 체크 리스트

  • 리뷰어 설정했나요?
  • Label 설정했나요?
  • 제목 양식 맞췄나요? (ex. feat: 기능 추가)
  • 변경 사항에 대한 테스트를 진행했나요?

Summary by CodeRabbit

  • Chores
    • Updated CORS configuration to support requests from a new development environment domain.

@coderabbitai
Copy link

coderabbitai bot commented Mar 13, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

A new allowed origin "https://dev.pinback.today" is added to the CorsConfig class, expanding the set of permitted cross-origin request sources. The change updates the CORS configuration without modifying control flow or error handling logic.

Changes

Cohort / File(s) Summary
CORS Configuration
api/src/main/java/com/pinback/api/config/cors/CorsConfig.java
Added "https://dev.pinback.today" to the allowed origins list for CORS policy.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

feat

Poem

🐰 A dev domain hops into place,
Dev.pinback.today joins the CORS race,
Cross-origin doors now wider spread,
With grateful hops from this bunny thread! 🌍

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description only contains the template with placeholder text and has not been filled in with actual implementation details. Fill in all required sections with meaningful content: PR summary/purpose, detailed changes made, precautions to note, and confirm completion of the checklist items.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'feat: cors 설정' correctly describes the main change, which is adding CORS configuration with a new allowed origin.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.

Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present.

@ose0221 ose0221 merged commit df39a3d into main Mar 13, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant