Security: Deleted content exposure + revision history IDOR via sequential IDs
- 主要言語
- Go
- スター
- 15.7k
- フォーク
- 1.4k
- 平均マージ
- 3日 8時間
- マージ済み PR(30日)
- 7
説明
## Summary
Two information disclosure issues allow access to deleted/pending content:
### 1. Deleted Answer Content via GET /answer/api/v1/answer/info (Low-Medium)
\`AnswerService.Get()\` calls \`answerRepo.GetByID()\` which fetches answers by raw ID with no \`status != deleted\` filter. Deleted answers are returned with full content to unauthenticated callers.
**Secure comparison:** \`SearchList\` correctly filters: \`session.Where("status != ?", entity.AnswerStatusDeleted)\`
### 2. Revision History IDOR via GET /answer/api/v1/activity/timeline/detail (Medium)
\`GetObjectTimelineDetail\` fetches revision content by sequential integer ID (\`autoincr BIGINT\`). No check for whether the associated question/answer is deleted or pending. Any authenticated user can enumerate \`revision_id=1,2,3...\` to read full content of deleted/pending questions and answers.
**Secure comparison:** \`GetRevisionList\` correctly checks \`objInfo.IsDeleted()\` before showing history. \`GetQuestion\` blocks access to deleted/pending content.
### Impact
- Bypasses content moderation (admin-deleted content remains accessible)
- Reveals pending-review content before publication
- Sequential revision IDs make enumeration trivial
## Discovery
Found through automated security research comparing content visibility controls across endpoints.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
影響を受ける2つのエンドポイントについて、まず AnswerService.Get と GetObjectTimelineDetail から始め、そのアクセスチェックを SearchList、GetRevisionList、GetQuestion と比較します。削除済みおよび保留中の質問、回答、リビジョンがどのように読み込まれるかを追跡します。未認証ユーザーに削除済みの回答が公開されず、認証済みユーザーにも削除済みまたは保留中のリビジョンの内容が公開されなくなり、許可されたコンテンツには引き続きアクセスできれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- go
- 領域
- api, backend, security
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100