Skip to content

Fix/5 dynamic gh hostname matching#20

Merged
jjeliga merged 2 commits intomainfrom
fix/5-dynamic-gh-hostname-matching
Apr 24, 2026
Merged

Fix/5 dynamic gh hostname matching#20
jjeliga merged 2 commits intomainfrom
fix/5-dynamic-gh-hostname-matching

Conversation

@jjeliga
Copy link
Copy Markdown

@jjeliga jjeliga commented Apr 23, 2026

Follow up to #5

The gateway gained a DYNAMIC_APP_PROVIDERS registry for providers whose host is resolved per-connection from metadata.baseUrl (vs. the static APP_PROVIDERS host rules), with GitHub Enterprise registered as the first such provider — git HTTPS paths get Basic auth with x-access-token, and /api/v3/* overrides to Bearer. Host matching in connect.rs was unified behind a new connection_matches_host helper that handles both static and dynamic providers, so the caller no longer needs to branch, and a typo of providers_for_host was removed from runtime code (kept only for tests).

jjeliga added 2 commits April 23, 2026 14:01
Signed-off-by: Jan Jeliga <jeliga.jan@gmail.com>
Signed-off-by: Jan Jeliga <jeliga.jan@gmail.com>
@jjeliga jjeliga merged commit 0a3d6d5 into main Apr 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants