Shuta-syd / 42-ft_transcendence

This project is about creating a website for the mighty Pong contest
1 stars 1 forks source link

【JWT】jwtのエラー解消、オンラインステータスの保持部分のリファクタ #271

Closed Shuta-syd closed 1 year ago

Shuta-syd commented 1 year ago

やったこと

動作確認

通常ログインもしくは42Loginをしたときにbackend側のログにオンラインになった旨が表示されるか

その他

issues

about : #264

close #264

public-jun commented 1 year ago

確かではないですが、accessTokenが期限切れなのにセットされたままの場合はjwtエラーが出ますがサービスとして挙動には問題ないと思います。

このエラーが解消されたわけではない?

public-jun commented 1 year ago

このPRに関係ないけど、42loginの場合はemailは設定できないんやっけ?

スクリーンショット 2023-04-22 17 39 57 スクリーンショット 2023-04-22 17 39 24
kny-i commented 1 year ago

@Shuta-syd 接続が切れてこれってoflineになった時には特に何も表示されないけど、問題ないかな?

Shuta-syd commented 1 year ago

このPRに関係ないけど、42loginの場合はemailは設定できないんやっけ?

スクリーンショット 2023-04-22 17 39 57 スクリーンショット 2023-04-22 17 39 24

42emailは変えられたらこまるので問題ないです

Shuta-syd commented 1 year ago

確かではないですが、accessTokenが期限切れなのにセットされたままの場合はjwtエラーが出ますがサービスとして挙動には問題ないと思います。

このエラーが解消されたわけではない?

自動的にログアウトする仕様を追加すれば大丈夫だと思うのですが、現在の仕様上バックエンドのログにはエラーログが表示されます。ですが、ユーザー側には問題ないです

Shuta-syd commented 1 year ago

@Shuta-syd 接続が切れてこれってoflineになった時には特に何も表示されないけど、問題ないかな?

これは実装していないから大丈夫