Skip to content

오다솔 백엔드 8주차 과제 제출합니다.#12

Open
fivedasol wants to merge 3 commits intomainfrom
week-8-custom
Open

오다솔 백엔드 8주차 과제 제출합니다.#12
fivedasol wants to merge 3 commits intomainfrom
week-8-custom

Conversation

@fivedasol
Copy link
Copy Markdown
Collaborator

login이랑 refresh 모두 잘 되는데, signup만 자꾸 401에러가 뜨는데 해결을 못했습니다..
image
image
image

@kseysh
Copy link
Copy Markdown
Member

kseysh commented May 27, 2024

아마 토큰 앞에 Bearer를 안붙여주셔서 그런 것 같아요!

Copy link
Copy Markdown
Member

@kseysh kseysh left a comment

Choose a reason for hiding this comment

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

id 중복 체크가 없는 것 같은데 맞나요?
그래도 심화과제까지 해보려고 한 모습 너무 좋습니다! 고생하셨어요~

Comment on lines -60 to -86
CORS_ALLOW_METHODS = [ # 허용할 옵션
'DELETE',
'GET',
'OPTIONS',
'PATCH',
'POST',
'PUT',
]

CORS_ALLOW_HEADERS = [ # 허용할 헤더
'accept',
'accept-encoding',
'authorization',
'content-type',
'dnt',
'origin',
'user-agent',
'x-csrftoken',
'x-requested-with',
]

CORS_ALLOW_CREDENTIALS = True

CORS_ALLOW_ORIGINS = [
'https://likelion-fivedasol.p-e.kr/',
]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

이건 지우신 이유가 있을까요 ?-?

Comment on lines +36 to +39
GENDER_OPT =(
('남자', '남자'),
('여자', '여자'),
)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

오 좋아요

Comment on lines +38 to +40
django_login(request, user)
refresh = RefreshToken.for_user(user)
update_last_login(None, user)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

만들어둔 login함수가 아닌 다른 함수로 사용한 이유가 있나요?
어떤 역할을 하는 함수인지 궁금해요

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