kstm-su / member-portal

kstmメンバーであることを確認し、また事務処理を簡潔化するためのポータルサイト
Apache License 2.0
1 stars 0 forks source link

OAuth2.0 サーバーの実装 #6

Closed Nlkomaru closed 2 months ago

nona-takahara commented 3 months ago

このPRでやりたいこと、やったこと、やらないことの要約を書いていただけますか?

他の部分の開発のためにも、どのようなエンドポイントが立つのかが知りたいです(もし参考になる文献があればそれを提示してもらえると助かります)

Cyndaquil1999 commented 2 months ago

仮置きのhtmlファイルなら大丈夫ですが、フロントがNext.js使ってるので、Next.jsのコンポーネントとか使った方がメンテしやすくて良いと思います!

Nlkomaru commented 2 months ago

仮置きのhtmlファイルなら大丈夫ですが、フロントがNext.js使ってるので、Next.jsのコンポーネントとか使った方がメンテしやすくて良いと思います!

public/view/authorize.htmlのことを指しているのであれば、これは認証ページのtemplateファイルなので、reactを使うのは重たいかなと思い、htmlに埋め込みのtailwindで書いています。

Nlkomaru commented 2 months ago

現在、public client (シークレットキーを保持しないクライアント)のみサポートしている状態で動作します。

Cyndaquil1999 commented 2 months ago

エンドポイントについてはclient_idなどの変数が必要そうなので未検証ですが、少なくとも400番を出しているので問題ないかと思います! 他は大丈夫だと思います。

今後の引継ぎ的な観点からすると、swaggerのようなAPIドキュメントを導入するのが良いと思いました!!