Skip to content

docs: make bounty forfeiture forward-looking, and fix the level 2 trigger - #22

Open
aljo242 wants to merge 1 commit into
mainfrom
docs/consequences-future-bounties
Open

aljo242 wants to merge 1 commit into
mainfrom
docs/consequences-future-bounties

Conversation

@aljo242

@aljo242 aljo242 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Two problems Matt and Eric found reading the blog draft.

Level 2 escalated on "publishing something an attacker can act on immediately", which is true of every disclosure, so it never distinguished anything from level 1.

Retroactively voiding a bounty on an already-submitted report is Immunefi's to enforce under their terms, not ours. Future eligibility is ours, so the consequences now run forward.

Not addressed here: the consequences still only bite downstream teams, not researchers who do not want a pre-disclosure seat.

…gger

Level 2 escalated on "a repeat instance, or publishing something an attacker can
act on immediately". Publishing vulnerability details is always something an
attacker can act on, so that clause caught every case and separated nothing from
level 1. A repeat instance is the trigger.

Forfeiting the bounty on a report already submitted runs through Immunefi under
their terms, so it is not ours to enforce. Future eligibility is. A warning
leaves an earned bounty alone, a suspension makes reports during those 3 months
ineligible, and a permanent block ends eligibility for good.

Raised by Matt and Eric reviewing the security blog draft.
@aljo242
aljo242 requested a review from a team as a code owner October 1, 2026 23:16
@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

[Low risk] Updates security policy language on bounty forfeiture.

The PR appears safe to merge, though the suspension wording should clarify how severity-based first-instance escalation works.

Findings

  1. P2 First-instance suspension is unclear ▶

Summary

The PR makes disclosure-related bounty consequences forward-looking and removes the immediately actionable publication trigger from level 2.

  • The revised level-2 wording needs to be reconciled with the existing severity-based escalation rule.

Reviews (1) · Last reviewed commit: "docs: make bounty forfeiture forward-loo..."

Comment thread SECURITY.md
Comment on lines +242 to +243
**2. Suspension.** A repeat instance removes the reporter from any private
notification or pre-disclosure list for 3 months. We continue to accept and act

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 First-instance suspension is unclear. Level 2 now names only a repeat instance as its trigger, while the policy below says severity can determine the level even for a first instance. That leaves the response team and reporters without a clear rule for a serious first disclosure that warrants suspension but not a permanent block. Please clarify whether severity can trigger a first-instance suspension.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant