-
Notifications
You must be signed in to change notification settings - Fork 23
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
7주차 미션 + 8주차 미션 / 서버 1조 박종익 #18
Open
bingbonbon
wants to merge
19
commits into
Konkuk-KUIT:main
Choose a base branch
from
bingbonbon:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
7주차 내용
50% 구현
어려웠던 점
구현한 것
궁금한 점
처음 yml 파일에 외부에서 민감한 데이터를 주입하는 방식으로 environmental variables를 써넣는 과정이 있었는데, 강의에서는 파트장님이 JWT-SECRET-KEY 부분에 해쉬값 같은 것을 집어넣는 것을 보았습니다.
그런데 저는 어떤 식으로 해야 할 지 몰라서 그냥 null로 집어넣고 진행했는데 어려움이 없어 그대로 진행했는데, 저 jwt는 어떤 식으로 생성했는지 궁금합니다!
지난 주차에서 REST api 설계만 하고 다른 예외 처리 부분은 강의에 나오지 않아서 배운 점들만 적용하여 코드를 짰습니다. 그런데 다른 사람들의 코드를 보니 BindingResult나 다른 예외 처리 부분을 사용하고 있어 내가 조금 부족했던 점이라고 생각하고 있었습니다. 하지만 이번 강의를 통해 예외 처리 부분을 배울 수 있었고, 좀 더 명확히 정보를 숙지한 후에 코드를 리팩토링할 수 있었습니다.
8주차 내용
구현한 점
궁금한 점
bindingResult가 정확히 어디까지 오류를 잡아주는지 궁금합니다! controller단에서 bindingResult를 파라미터로 제공하면, controller가 호출하는 service와 그 아래 DAO나 DTO에서 잡히는 예외 처리까지 한번에 다 담아두는 지 아니면 명시가 된 컨트롤러에서 잡는 @validated에 걸리는 예외만 잡는지 궁금합니다!