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.

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

Introduce the notion of a default SessionProtocol #6128

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

minor fixes

f9cb6bd
Select commit
Loading
Failed to load commit list.
Open

Introduce the notion of a default SessionProtocol #6128

minor fixes
f9cb6bd
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Feb 27, 2025 in 1s

84.09% of diff hit (target 74.44%)

View this Pull Request on Codecov

84.09% of diff hit (target 74.44%)

Annotations

Check warning on line 34 in core/src/main/java/com/linecorp/armeria/internal/client/SessionProtocolUtil.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

core/src/main/java/com/linecorp/armeria/internal/client/SessionProtocolUtil.java#L33-L34

Added lines #L33 - L34 were not covered by tests

Check warning on line 114 in eureka/src/main/java/com/linecorp/armeria/client/eureka/EurekaEndpointGroup.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eureka/src/main/java/com/linecorp/armeria/client/eureka/EurekaEndpointGroup.java#L114

Added line #L114 was not covered by tests

Check warning on line 155 in eureka/src/main/java/com/linecorp/armeria/client/eureka/EurekaEndpointGroup.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eureka/src/main/java/com/linecorp/armeria/client/eureka/EurekaEndpointGroup.java#L155

Added line #L155 was not covered by tests

Check warning on line 89 in eureka/src/main/java/com/linecorp/armeria/server/eureka/EurekaUpdatingListener.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eureka/src/main/java/com/linecorp/armeria/server/eureka/EurekaUpdatingListener.java#L89

Added line #L89 was not covered by tests

Check warning on line 130 in eureka/src/main/java/com/linecorp/armeria/server/eureka/EurekaUpdatingListener.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

eureka/src/main/java/com/linecorp/armeria/server/eureka/EurekaUpdatingListener.java#L130

Added line #L130 was not covered by tests

Check warning on line 149 in grpc/src/main/java/com/linecorp/armeria/client/grpc/GrpcClients.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

grpc/src/main/java/com/linecorp/armeria/client/grpc/GrpcClients.java#L149

Added line #L149 was not covered by tests

Check warning on line 218 in grpc/src/main/java/com/linecorp/armeria/client/grpc/GrpcClients.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

grpc/src/main/java/com/linecorp/armeria/client/grpc/GrpcClients.java#L218

Added line #L218 was not covered by tests

Check warning on line 96 in thrift/thrift0.13/src/main/java/com/linecorp/armeria/client/thrift/ThriftClients.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

thrift/thrift0.13/src/main/java/com/linecorp/armeria/client/thrift/ThriftClients.java#L96

Added line #L96 was not covered by tests

Check warning on line 184 in thrift/thrift0.13/src/main/java/com/linecorp/armeria/client/thrift/ThriftClients.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

thrift/thrift0.13/src/main/java/com/linecorp/armeria/client/thrift/ThriftClients.java#L184

Added line #L184 was not covered by tests

Check warning on line 277 in thrift/thrift0.13/src/main/java/com/linecorp/armeria/client/thrift/ThriftClients.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

thrift/thrift0.13/src/main/java/com/linecorp/armeria/client/thrift/ThriftClients.java#L277

Added line #L277 was not covered by tests