Skip to content

Fix bare IPv6 host matching in config#630

Merged
ryanfowler merged 1 commit into
mainfrom
fix-bare-ipv6-config-matching
May 27, 2026
Merged

Fix bare IPv6 host matching in config#630
ryanfowler merged 1 commit into
mainfrom
fix-bare-ipv6-config-matching

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Normalize bare URLs with bracketed IPv6 authorities before host matching so config host sections use the same host form as Url::host_str().
  • Treat [::1]:3000/path as host ::1 instead of splitting on the first colon.
  • Add a config regression test covering a [::1] host section applied to a bare IPv6 URL.

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • Added a focused unit test for the IPv6 host-matching regression.

@ryanfowler ryanfowler enabled auto-merge May 27, 2026 22:09
@ryanfowler ryanfowler merged commit 527e883 into main May 27, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-bare-ipv6-config-matching branch May 27, 2026 22:10
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