Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Hotfix-05-인증-aop-정상화] #102

Open
wants to merge 4 commits into
base: 047-MemberLocation-기본값-불변객체
Choose a base branch
from

Conversation

ScottSung7
Copy link
Collaborator

변경사항

  • 조금씩 건드려 놨던 로컬 파일들을 시간되는 한 마무리 중입니다. 커밋 패널에서 최근 수정사항을 찾기가 조금 어려워져 DDD로 구조 변경을 하며 한번에 하나씩 하는 습관을 가지려고 수정되어 있는 것들을 시간날때 조금씩 마무리하려고 합니다.
  • 이후 필터로 변경할까 하고 있지만 현재까지 AOP로 로그인 인증을 확인하고 있었는데 적용되고 있지 않아서 수정하였습니다.
  • RestController 태그를 붙여주어야 하는데 Controller로 붙여 놓은 것들이 있어서 변경하였습니다.
    • String을 주는 경우에는 오류가 났었는데 스프링 타임리프 라이브러리를 안 지웠나라고 생각했는데 이거였군요..
    • ResponseEntity를 내려주는 경우에는 다행히 제대로 동작해서 문제가 없었던 것 같습니다. 확실히 하기위해 Member MS에서는 모두 변경했습니다. 다른 MS도 수정사항때 변경하겠습니다!

@ScottSung7 ScottSung7 self-assigned this Nov 1, 2024
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