Skip to content

ci/security: enforce exact-head required checks for admins before merge #49

Description

@reprewindai-dev

Finding

Canonical main is protected and requires status context verify, but GitHub reports the required-status-check enforcement level as non_admins. That leaves an administrator bypass path even while exact-head cAPI CI jobs are currently being created without runners/steps and concluding failure.

This is a merge-control defect, not evidence that current remediation source failed tests.

Required correction

  • enforce the required exact-head verify gate for administrators as well as non-admins;
  • require blocking review state to be respected for all merge actors;
  • ensure the verify workflow actually receives a runner and executes test/lint/build/security checks;
  • run a clean exact-head validation on current default branch after Actions execution is restored;
  • add/record a negative administrative merge test proving a failed or non-executed required check cannot be overridden.

Truth boundary

Do not promote source-observed changes or PR prose to VERIFIED because branch protection exists. Current runtime remains NOT_VERIFIED until deployed SHA + HTTP/protocol identity + listener + Traefik routing agree.

This issue is intentionally separate from #44's fail-closed authority code fix; #44 must remain draft until the executable gate actually passes.

Activity

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

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions