Skip to content

drive-foundation follow-up: レビュー nit 集約 (バリデーション・パフォーマンス・堅牢化) #3

Closed

Description

@turtton

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

  1. turtton commented on Sep 6, 2026

    @turtton
    MemberAuthor

    intent-cli packet として正式に issue 化されました → #4 (drive-foundation-followups)。本 issue は superseded として close します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions