PR #2 (drive-foundation) のコードレビューで検出された nit 級指摘の集約。機能阻害なし。 - ファイル名 (UploadQuery.name) に長さ検証なし (フォルダ名は 255 字検証あり) - files.rs の DefaultBodyLimit::max(...) レイヤは Body 直接抽出ハンドラに実質 no-op (実強制は CountingBody。誤解を招くので整理したい) - 課金解決が全認証リクエストで DB 2-3 クエリ (global rules + owner rules + plan)。キャッシュなし - RateLimiters はプロセス内メモリ (複数レプリカ非共有)。rpm 変更で limiter 再生成 → burst リセット - rules.rs の upsert_rule 2 段 upsert は同時 INSERT で unique violation → 500 - ensure_owned_folder の事前チェックと INSERT が非原子的 (競合で FK 違反 500。404/409 が望ましい) - /public/{key} は無認証・レート制限なし (帯域計量は初動外だが DoS 面で記録) - immutable キャッシュ設計により unpublish 後もブラウザ/CDN に残る可能性 (仕様として文書化) 元レビュー: code-reviewer subagent, 2026-08-30, PR #2 review ラウンド
Activity
turtton commentedon Sep 6, 2026
intent-cli packet として正式に issue 化されました → #4 (drive-foundation-followups)。本 issue は superseded として close します。