Skip to content

docs(gha init): licence lookups keep 80% of the job token's quota and prior scanner steps get no token - #23

Merged
0x73746F66 merged 1 commit into
mainfrom
docs/license-lookup-quota
Nov 13, 2026
Merged

0x73746F66 merged 1 commit into
mainfrom
docs/license-lookup-quota

Conversation

@0x73746F66

Copy link
Copy Markdown
Member

Documents Vulnetix/cli#342 in cli/ci-cd/granular-actions.md.

  • Scanners skip impossible names and stop early. They never ask GitHub about a name that cannot be a repository (uses: $/sbom, globs, placeholders), and a repository 404 ends the lookup, so one repository costs 1–4 requests.
  • In GitHub Actions the lookups keep 80% of the job token's hourly limit. They stop at 4,000 of 5,000, above analyze's half. They also stop on any rate-limit answer, and unchecked licences stay UNKNOWN with a warning.
  • Each batch logs its request count, so run logs show what each job spent.
  • Prior steps get no token. An extra's prior scanner step gets github-token: "" on every event.
  • Includes the evidence. On the push that merged fix(analyze): a spent GitHub quota at preflight skips the forge pass instead of failing the run cli#341 the scan jobs' lookups spent 4,913 requests in under 3.5 minutes, and the SCA job's SARIF upload failed on the rate limit.

… prior scanner steps get no token

Scanners never ask GitHub about a name that cannot be a repository and stop at a repository 404. Inside GitHub Actions they stop at 4,000 of 5,000 requests left, above analyze's half, and on any rate-limit answer, leaving unchecked licences UNKNOWN and logging what each batch spent. An extra's prior scanner step gets github-token "" on every event. Matches Vulnetix/cli#342.
@0x73746F66
0x73746F66 merged commit 6433405 into main Nov 13, 2026
17 checks passed
@0x73746F66
0x73746F66 deleted the docs/license-lookup-quota branch November 13, 2026 17:44
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