Skip to content

xds: Add support for RFC 3986 URIs#12660

Merged
jdcormie merged 2 commits into
grpc:masterfrom
jdcormie:grpc-uri-xds
Mar 16, 2026
Merged

xds: Add support for RFC 3986 URIs#12660
jdcormie merged 2 commits into
grpc:masterfrom
jdcormie:grpc-uri-xds

Conversation

@jdcormie

Copy link
Copy Markdown
Member

No description provided.

@shivaspeaks shivaspeaks left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM.

@jdcormie jdcormie merged commit a43f387 into grpc:master Mar 16, 2026
17 checks passed
jdcormie added a commit to jdcormie/grpc-java that referenced this pull request Mar 24, 2026
Fixes an oversight in grpc#12660. io.grpc.Uri#getAuthority makes a
distinction between xds:///service and xds:/service but java.net.URI
does not. XdsNameResolver wants to treat those two cases the same.
jdcormie added a commit to jdcormie/grpc-java that referenced this pull request Mar 24, 2026
Fixes an oversight in grpc#12660. io.grpc.Uri#getAuthority makes a
distinction between xds:///service and xds:/service but java.net.URI
does not. XdsNameResolver wants to treat those two cases the same.
jdcormie added a commit that referenced this pull request Mar 25, 2026
Fixes an oversight in #12660. io.grpc.Uri#getAuthority makes a
distinction between xds:///service and xds:/service but java.net.URI
does not. XdsNameResolver wants to treat those two cases the same.
kannanjgithub pushed a commit to kannanjgithub/grpc-java that referenced this pull request May 4, 2026
kannanjgithub pushed a commit to kannanjgithub/grpc-java that referenced this pull request May 4, 2026
Fixes an oversight in grpc#12660. io.grpc.Uri#getAuthority makes a
distinction between xds:///service and xds:/service but java.net.URI
does not. XdsNameResolver wants to treat those two cases the same.
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants