Skip to content

[codex] Improve URI authority and query handling - #33

Merged
ntoskrnl7 merged 1 commit into
masterfrom
codex/improve-uri-authority-query
Jun 15, 2026
Merged

ntoskrnl7 merged 1 commit into
masterfrom
codex/improve-uri-authority-query

Conversation

@ntoskrnl7

Copy link
Copy Markdown
Owner

Summary

  • Split userinfo@host into a new basic_uri::userinfo field and parse bracketed IPv6 host literals without treating IPv6 colons as ports.
  • Validate ports as digit-only decimal values in the 0..65535 range, making malformed ports invalidate the URI.
  • Add query_params() and decode_component() for parsed, percent-decoded query access.
  • Percent-encode query_map and u8query_map keys/values when appending them to a URI, preserving existing query/fragment placement.
  • Add RFC-style relative reference resolution with dot-segment path cleanup through resolve() and remove_dot_segments().
  • Update URI docs and extend narrow/wide tests for the new behavior.

Verification

  • git diff --check
  • /tmp/cmake-3.30.5/bin/cmake -S test -B /tmp/ntoskrnl7-ext-build-uri-advanced -DCMAKE_BUILD_TYPE=Release
  • /tmp/cmake-3.30.5/bin/cmake --build /tmp/ntoskrnl7-ext-build-uri-advanced --config Release -j 2
  • /tmp/cmake-3.30.5/bin/ctest --test-dir /tmp/ntoskrnl7-ext-build-uri-advanced -C Release --output-on-failure

@ntoskrnl7
ntoskrnl7 marked this pull request as ready for review June 15, 2026 11:07
@ntoskrnl7
ntoskrnl7 merged commit 3e12e13 into master Jun 15, 2026
8 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.

1 participant