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

[BE] 전시현 로그인 #5

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Conversation

jj0526
Copy link

@jj0526 jj0526 commented May 16, 2024

미션 실행 실패 케이스 존재 시, 실패한 이유 고찰해보기

image
image
token을 생성하는 부분에서 오류가 난 거 같습니다

image
image

로그인 부분은 구현 했지만 attend하는 부분은 실행이 안됩니다

@jj0526 jj0526 self-assigned this May 16, 2024

private String makeToken(Date expiry, User user) {
Date now = new Date();
String encodedEmail = Base64.getEncoder().encodeToString(user.getEmail().getBytes());
Copy link
Member

Choose a reason for hiding this comment

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

해당 오류는 base64를 인코딩하는 과정에서 발생하는 문제 같습니다.
base64에서 원칙적으로 사용하지 않는 문자를 인코딩 과정에서 포함될 경우 해당 오류가 나타날 수 있습니다.
safeencoding을 사용하면 위와 같은 문제를 해결 할 수 있을 것 같습니다!

String encodedEmail = Base64.getUrlEncoder().encodeToString(user.getEmail().getBytes());

참고해주세요

Comment on lines 21 to 25
issuer: [email protected]
secret_key: study-springboot
header: Authorization
secret: 57e28d84f356b2af66d4fbd5d38efece93455f1b0fac02a28f4be437d9172462
token-validity-in-seconds: 86400
Copy link
Member

Choose a reason for hiding this comment

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

key는 환경변수로 따로 관리해주는게 좋을것 같습니다!

Comment on lines 6 to 9
url: jdbc:h2:mem:testdb
MODE: MySQL
username: sa
generate-unique-name: false
Copy link
Member

Choose a reason for hiding this comment

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

이번에는 h2를 사용했는데 로컬에서 mysql사용해보는걸 연습해보는건 어떨까요?
추후 프로젝트에서 로컬 db를 사용하게 될 때 mysql을 많이 사용해서 적용해보시면 좋을 것 같습니다

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.

2 participants