coder13 / coder13/LetsCube

Rate-limit authentication endpoints

オープン
#239 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
30
フォーク
9
PR マージ指標
30日以内にマージされた PR はありません

説明

Follow-up slice of #227.

Apply the established API rate-limit policy to the authentication router, which is outside the existing /api limiter.

Scope:
- protect authentication code exchange and logout routes consistently;
- add a focused regression test for requests exceeding the configured IP window;
- retain the current response contract and CSRF protection;
- do not add, inspect, store, or search email data.

Acceptance criteria:
- repeated authentication-route requests receive the standard 429 response;
- normal sign-in and logout behavior still works within the limit;
- server tests and CodeQL pass.

Production rollout validation remains part of #176.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず、フォローアップ issue #227 に記載された既存の API レート制限ポリシーを読み、認証ルーター、特にコード交換とログアウトのルートを見つけます。設定された IP ウィンドウを超えるリクエストに対するサーバーテストのカバレッジを重点的に追加し、その後、標準の 429 レスポンス、既存のサインイン/ログアウトの動作、CSRF 保護、CodeQL の結果を、レスポンス契約を変更せずに検証します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, nodejs
領域
api, authentication, backend, security
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
55/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。