Skip to content

fix(asf-security-dash): refuse non-ASF projects - #1798

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix/asf-dash-asf-only
Oct 25, 2026
Merged

potiuk merged 1 commit into
apache:mainfrom
potiuk:fix/asf-dash-asf-only

Conversation

@potiuk

@potiuk potiuk commented Oct 25, 2026

Copy link
Copy Markdown
Member

dash.security.apache.org only tracks ASF projects, but asf-security-dash and the security-issue-sync dashboard section ran for any adopter. For a non-ASF project they would read the dashboard cookie and call an ASF service that knows nothing about the project.

Changes

  • asf-security-dash CLI: a new required --project-md argument. The command exits with code 3 (NotAsf) unless the project's organization is exactly ASF. The check runs before the cookie is read or any request is made.
    • The value comes from the Identity table row, or an organization: line. HTML comments and fenced code are ignored, and the value must match in full, so ASFX, ASF-fork or a commented-out ASF row do not count. An annotated value such as `ASF` (derived: apache/ GitHub org) is accepted. A missing value or TODO counts as independent.
  • issue-sync dashboard section: a new Step D0 makes the section ASF-only. For a project whose upstream is under apache/, it suggests setting organization: ASF with /magpie-setup:setup config. D1 and D3 pass --project-md and handle exit 3.
  • Eval: security-issue-sync/step-d0-asf-only covers three cases: an ASF project, an independent project, and a TODO organization.

Testing

🤖 Generated with Claude Code

The dash.security.apache.org dashboard only tracks ASF projects. The
CLI now takes a required --project-md, and it exits 3 (NotAsf) unless
the organization there is exactly ASF. The check runs before the
cookie or the network are touched. HTML comments and fenced examples
are ignored when reading the value.

The issue-sync dashboard section gains Step D0 to match, with an eval
covering ASF, independent and TODO organizations.

Generated-by: Claude Opus 5
@potiuk
potiuk enabled auto-merge (squash) October 25, 2026 00:13
@github-actions github-actions Bot added family:security security-* skills family:tools tools/* family:docs Docs, MISSION.md, READMEs capability:intake Import external signal + sync tracker state substrate:framework-dev Tool substrate: build / validate / eval the framework itself contract:security-tracker Tool capability: security-tracker backend for embargoed reports (GitHub / files / Jira) labels Oct 25, 2026
@potiuk
potiuk merged commit c7c4e3d into apache:main Oct 25, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

capability:intake Import external signal + sync tracker state contract:security-tracker Tool capability: security-tracker backend for embargoed reports (GitHub / files / Jira) family:docs Docs, MISSION.md, READMEs family:security security-* skills family:tools tools/* substrate:framework-dev Tool substrate: build / validate / eval the framework itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant