Skip to content

Preserve canonical GitHub owner casing - #11

Merged
patcGM merged 1 commit into
mainfrom
fix/github-owner-case
Oct 22, 2026
Merged

patcGM merged 1 commit into
mainfrom
fix/github-owner-case

Conversation

@patcGM

@patcGM patcGM commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Fix

  • preserve mixed-case GitHub owner names when parsing canonical execution URIs
  • avoid urllib.parse.hostname lowercasing that rejected the real patcGM/closurectl-lab-v1 target
  • retain exact URI equality and all existing credential, port, path, query, and fragment checks

Live evidence

The protected exact-main-SHA provider preview failed closed before mutation with:

GitHub execution target must be canonical: github://patcgm/...

No GitHub or npm mutation was attempted.

Verification

  • regression test for mixed-case owner and repository names
  • 103 tests; 76% branch coverage
  • Ruff, format, actionlint, Bandit, and pip-audit pass

The provider drill will be restarted only after this fix is reviewed, merged, and exact-main-SHA CI succeeds.

@patcGM
patcGM requested a review from test2GM as a code owner October 22, 2026 11:54
@patcGM
patcGM merged commit 9f8794d into main Oct 22, 2026
9 checks passed
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.

2 participants