fix(mstsgu): default gateway port to HTTPS - #1823
Open
Marc-André Moreau (mamoreau-devolutions) wants to merge 2 commits into
Open
fix(mstsgu): default gateway port to HTTPS#1823Marc-André Moreau (mamoreau-devolutions) wants to merge 2 commits into
Marc-André Moreau (mamoreau-devolutions) wants to merge 2 commits into
Conversation
Copilot started reviewing on behalf of
Marc-André Moreau (mamoreau-devolutions)
August 28, 2026 21:25
View session
Marc-André Moreau (mamoreau-devolutions)
deployed
to
llm-providers
August 28, 2026 21:25 — with 
GitHub Actions
Active
GitHub Actions
Active
github-actions
Bot
added
maintainer-required
Contributor
There was a problem hiding this comment.
Pull request overview
Defaults host-only RD Gateway endpoints to HTTPS port 443 while preserving explicit ports and IPv6 formatting.
Changes:
- Parses and normalizes gateway authorities.
- Adds endpoint validation and normalization tests.
- Documents optional gateway ports.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/packet_io.rs |
Implements endpoint parsing and default port handling. |
src/test_support.rs |
Exposes normalization for tests. |
tests/proxy.rs |
Tests valid and malformed endpoints. |
src/lib.rs |
Documents endpoint syntax. |
Marc-André Moreau (mamoreau-devolutions)
deployed
to
llm-providers
August 29, 2026 02:00 — with
GitHub Actions
Active
GitHub Actions
Active
Accept host-only gateway endpoints as HTTPS authorities and connect to port 443. Explicit ports and bracketed IPv6 literals retain their normalized endpoints.
Reject userinfo in gateway endpoints before parsing their authority so malformed targets cannot select an unintended host.
Marc-André Moreau (mamoreau-devolutions)
force-pushed
the
copilot/gateway-default-port
branch
from
August 29, 2026 13:36
66c1226 to
3125b1b
Compare
Marc-André Moreau (mamoreau-devolutions)
deployed
to
llm-providers
August 29, 2026 13:36 — with
GitHub Actions
Active
GitHub Actions
Active
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Accept host-only gateway endpoints as HTTPS authorities and connect to port 443. Explicit ports and bracketed IPv6 literals retain their normalized endpoints.