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

認証周りの動作を改善 #55

Open
riii111 opened this issue Oct 21, 2024 · 2 comments
Open

認証周りの動作を改善 #55

riii111 opened this issue Oct 21, 2024 · 2 comments
Assignees
Labels

Comments

@riii111
Copy link
Owner

riii111 commented Oct 21, 2024

  • エラークラスを一律raiseしているので、401が返ってきた時、Unhandled Runtime Errorが起こってしまうんで操作性が悪い

  • APIを実行した結果、401エラーの場合

    • トークン更新APIを実行する
      • 循環参照が起こらないように、customFetchは使わないこと
      • Cookieベースでチェックしてもらうこと
      • それでも失敗した場合は/authへリダイレクトさせること

注意点

  • クライアントサイド、サーバーサイドで細かいロジックが変わるので注意
  • middlewareとcore両方で修正が必要か?
@riii111 riii111 added enhancement New feature or request frontend backend labels Oct 21, 2024
@riii111 riii111 self-assigned this Oct 21, 2024
@riii111
Copy link
Owner Author

riii111 commented Oct 21, 2024

うまくいかん。Middlewareかroute handlerでやるべき?

  1. ページを開く(サーバーサイドのフェッチが実行される)
  2. トークンが無効なら、401が返ってくる
  3. credentials: "include" でトークン更新APIを実行(その後に実行したかったAPIを再リクエストする。一回だけ)
    →しかしCookieを付与した状態でリクエストできていない模様

@riii111
Copy link
Owner Author

riii111 commented Oct 23, 2024

memo

Cookie付与してリクエストし、トークン更新まではできた。
けど、その後のフェッチで新しいトークンを付与してリクエストできず。
その他、Route Handlerかましてリフレッシュ&リトライフェッチ実装したけどそれでもうまく反映されなかった。

riii111 added a commit that referenced this issue Oct 23, 2024
401の場合、トークン更新API実行→新しいアクセストークン使ってリトライし、それも失敗したら認証画面へ
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant