Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[1/3 header changes][metrics + grpc] Emit metrics for new headers handling behaviour. Hide actual changes under feature flag. #2268

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Renamed headerCallerProcedureToRequest to transportHeadersToRequest

d8e451e
Select commit
Loading
Failed to load commit list.
Open

[1/3 header changes][metrics + grpc] Emit metrics for new headers handling behaviour. Hide actual changes under feature flag. #2268

Renamed headerCallerProcedureToRequest to transportHeadersToRequest
d8e451e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jul 30, 2024 in 0s

92.21% of diff hit (target 80.00%)

View this Pull Request on Codecov

92.21% of diff hit (target 80.00%)

Annotations

Check warning on line 56 in transport/tchannel/response_writer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

transport/tchannel/response_writer.go#L55-L56 

Added lines #L55 - L56 were not covered by tests

Check warning on line 76 in transport/tchannel/response_writer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

transport/tchannel/response_writer.go#L76 

Added line #L76 was not covered by tests

Check warning on line 103 in transport/tchannel/response_writer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

transport/tchannel/response_writer.go#L103 

Added line #L103 was not covered by tests

Check warning on line 112 in transport/tchannel/response_writer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

transport/tchannel/response_writer.go#L112 

Added line #L112 was not covered by tests

Check warning on line 121 in transport/tchannel/response_writer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

transport/tchannel/response_writer.go#L121 

Added line #L121 was not covered by tests

Check warning on line 137 in transport/tchannel/response_writer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

transport/tchannel/response_writer.go#L137 

Added line #L137 was not covered by tests
View more details on Codecov