search

swsnu / swppfall2020

28 stars 17 forks source link

[HW4] the authentication info should be changed properly #186

Open cheezafizz opened 4 years ago

cheezafizz commented 4 years ago

안녕하세요 조교님, 과제를 하다가 궁금한 점이 생겨서 질문드립니다.

api/signin과 api/signout의 조건을 보면

the authentication info should be changed properly

라는 표현이 있습니다.

조교님께서 제시하신 표현을 기준으로 과제를 해결하려는데

이 표현이 정확히 무엇을 의도한 것인지 알 수가 없어서 질문드립니다.

위 표현은 아래 중 어느 것을 의도한 것인가요?

  1. authenticate 및 login또는 logout 함수를 사용하여라

  2. authentication info는 is_authenticated를 의미하며, 이 값이 적절히 바뀌는지 확인하여라

  3. request.user의 is_authenticated 값은 변하지 않으므로 직접 바꿔주어라

아무리 고민해보아도 모호한 점이 많아서 질문드립니다

kooyunmo commented 4 years ago
  • POST api/signin: Authenticate with the information given by request JSON body. If success, log-in (the authentication info should be changed properly) and response with 204. If fail, response with 401.
  • GET api/signout: If the user is authenticated, log-out (the authentication info should be changed properly) and response with 204. If not, response with 401.

해당 부분에 대해 너무 많은 생각을 하실 필요는 없습니다. 말 그대로 signin / signout을 했을 때 user에 대해서 is_authenticated 메서드를 호출했을 때 리턴되는 auth 값이 제대로 변하도록 하라는 의미입니다. 특별히 edge case 처리를 하라는 의미가 아니라 적절히 login status를 바꾸라는 의미일 뿐입니다.

cheezafizz commented 4 years ago

늦은 시간에도 빠른 답변 감사합니다.

jaewooklee93 commented 4 years ago

저는 1의 의미로 과제를 수행하였습니다. (+2도 맞는 말인 것 같습니다) 3번처럼 is_authenticated에 직접 값을 대입해줄 필요는 아마 없지 않을까 생각됩니다.