#18840
89%

Ran 22 Sep 2023 03:40PM UTC

Jobs 1

Files 543

Run time 19s

Badge

Embed ▾

Committed 22 Sep 2023 03:25PM UTC coverage: 88.279%. Remained the same

Build # #18840

Build Type

push

github-actions

Committed by

web-flow

Commit Message

Add fix for xdstp replacement for encoded authorities (#10571)

In ac35ab6 the logic in xDS Name resolver was changed to support encoded
authorities. This seems to cause an issue for xdstp replacements which
would percent encode the authority for the replacement causing double
encoding.

For example:

URI = xds:///path/to/service
Authority = path%2Fto%2Fservice
xdstp resource = xdstp:///envoy.config.listener.v3.Listener/path%252Fto%252Fservice

Here the authority is encoded due to slashes and during replacement we
percent encode it again causing %2F to change to %252F. To avoid this
issue, use the encoded authority only for the getServiceAuthority() API
and for all other use cases retain the unencoded authority.

Run Details

30383 of 34417 relevant lines covered (88.28%)

0.88 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	#18840.1	22 Sep 2023 03:40PM UTC	0	88.28

Source Files on build #18840

Detailed source file information is not available for this build.

grpc / grpc-java / #18840
89%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build #18840

grpc / grpc-java / #18840 89%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build #18840

grpc / grpc-java / #18840
89%

README BADGES
x