Skip to content

Conversation

@kjhh2605
Copy link
Contributor

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@kjhh2605 kjhh2605 linked an issue Aug 19, 2025 that may be closed by this pull request
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary of Changes

Hello @kjhh2605, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 저장하기(북마크) 기능의 오류를 수정하고, CORS(Cross-Origin Resource Sharing) 설정을 강화합니다. 북마크 삭제 시 잘못된 사용자 ID로 북마크를 찾는 문제를 해결하고, CORS 허용 오리진, HTTP 메서드 및 헤더를 보다 구체적으로 제한하여 보안을 개선합니다.

Highlights

  • 북마크 삭제 로직 수정: 북마크 삭제 시 사용자 ID를 하드코딩된 값(1L) 대신 실제 로그인한 사용자의 ID를 사용하도록 수정하여, 올바른 북마크를 찾고 삭제할 수 있도록 합니다.
  • CORS 설정 강화: CORS 허용 오리진을 특정 도메인(https://eatpic.shop)으로 제한하고, 허용되는 HTTP 메서드에서 OPTIONS를 제거하며, 허용되는 헤더를 Authorization과 Content-Type으로 명시하여 보안을 강화합니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

안녕하세요. 저장하기 관련 오류를 수정하는 PR을 리뷰했습니다. BookmarkServiceImpl.java에서 북마크 삭제 시 하드코딩된 사용자 ID를 사용하던 버그를 올바르게 수정한 점이 좋습니다. 이와 유사한 로직이 적용된 다른 부분도 함께 점검해보시면 더 좋을 것 같습니다. CorsConfig.java에서는 보안 강화를 위해 CORS 설정을 더 엄격하게 변경하셨네요. 좋은 변화입니다. 다만, 개발 환경 등 다양한 환경을 고려하여 허용할 출처(origin)를 설정 파일에서 관리하도록 개선하는 것을 제안합니다. 관련하여 상세한 리뷰 의견을 남겼으니 확인 부탁드립니다.

@2anizirong 2anizirong assigned 2anizirong and kjhh2605 and unassigned 2anizirong Aug 19, 2025
@2anizirong 2anizirong added the 🐛 Bug 버그가 발생했을 때 label Aug 19, 2025
@2anizirong 2anizirong self-requested a review August 19, 2025 10:14
Copy link
Member

@2anizirong 2anizirong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고했어용

@2anizirong 2anizirong changed the title [Fix] 저장하기 오류 수정 Fix: 저장하기 오류 수정 Aug 19, 2025
@2anizirong 2anizirong merged commit ebeec13 into main Aug 19, 2025
2 checks passed
@2anizirong 2anizirong deleted the fix/167-bookmark branch August 19, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Bug 버그가 발생했을 때

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] 카드 저장기록 삭제 오류

3 participants