-
Notifications
You must be signed in to change notification settings - Fork 0
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
[PDS-35] 아카이빙 자료 업로드 API 기능구현 #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셧슴다 리뷰 확인 부탁
String token = webRequest.getHeader(Constants.JWT.AUTHORIZATION_HEADER); | ||
if(!StringUtils.hasText(token)) throw new BaseException(BaseResponseCode.NULL_TOKEN); | ||
// 추출 | ||
token = token.replace(Constants.JWT.BEARER_PREFIX, ""); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 썼다가 에러 나서 replace 대신에 subString으로 변경해주세유
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replace를 썼을 때 에러나는 이유가 무엇인가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지난번 다음과 같은 에러를 뱉어내어 확인해보니 파싱할 때 대체로 subString을 사용하더라구요
nested exception is io.jsonwebtoken.io.DecodingException: Illegal base64url character: ' ']
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다
secret: ENC(7JHqgwmXfXySGksMK5SOUrWg7edAhDXA044D8pyPbQbsYgdgs+by47ti/tHUwlwqSCMmmD9FM3tZvazmb/0wtqTKqYXGsrNnlkD2yFO11Z6Gis49IO0SWhaAnmyUexlwy+Bscc2pZBK12M50vCeHkFQGRIe1SlPj) | ||
|
||
aws: | ||
s3: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
추후에 WAS에 s3 빼봅시다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고독한아카이빙 수고하셨습니단
.project(project) | ||
.user(user) | ||
.build(); | ||
project.addMaterial(material); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
project.addMaterial(material);
이 코드는 어떤 로직이에여 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 양방향매핑 때문이면 별도로 추가 안 해줘도 알아서 조회하는걸로 알고 있는디 !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
프로젝트와 자료 연관관계 맺는 로직인데 수정이 필요할것같아요!
👾 작업 내용
📸 스크린샷
🎸 기타 사항