Skip to content

Add unary-first SDK-native gRPC client transport on Restlet - #1563

Merged
jlouvel merged 4 commits into
3.0from
feat/rpc-r3-grpc
Oct 31, 2026
Merged

jlouvel merged 4 commits into
3.0from
feat/rpc-r3-grpc

fix: apply the gRPC call duration cap after application interceptors

ef68447
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Oct 31, 2026 in 1m 9s

Annotations

Check failure on line 202 in org.restlet.ext.grpc/src/main/java/org/restlet/ext/grpc/internal/RestletClientStream.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 22 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8zsRd1Q3-BF5g5-&open=AaFrU8zsRd1Q3-BF5g5-&pullRequest=1563

Check warning on line 553 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g51&open=AaFrU8znRd1Q3-BF5g51&pullRequest=1563

Check warning on line 303 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcHttpValidationTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()".

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaGQrvuwwmll2o7CsM4Y&open=AaGQrvuwwmll2o7CsM4Y&pullRequest=1563

Check warning on line 581 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g52&open=AaFrU8znRd1Q3-BF5g52&pullRequest=1563

Check warning on line 539 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g50&open=AaFrU8znRd1Q3-BF5g50&pullRequest=1563

Check warning on line 101 in org.restlet.ext.grpc/src/main/java/org/restlet/ext/grpc/internal/RestletClientStream.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use "Math.clamp" instead of "Math.min" or "Math.max".

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8zsRd1Q3-BF5g58&open=AaFrU8zsRd1Q3-BF5g58&pullRequest=1563

Check warning on line 593 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g53&open=AaFrU8znRd1Q3-BF5g53&pullRequest=1563

Check warning on line 63 in org.restlet.ext.grpc/src/main/java/org/restlet/ext/grpc/internal/RestletClientStream.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Constructor has 10 parameters, which is greater than 7 authorized.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8zsRd1Q3-BF5g56&open=AaFrU8zsRd1Q3-BF5g56&pullRequest=1563

Check failure on line 95 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/internal/RestletClientStreamTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8ziRd1Q3-BF5g5u&open=AaFrU8ziRd1Q3-BF5g5u&pullRequest=1563

Check warning on line 667 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g54&open=AaFrU8znRd1Q3-BF5g54&pullRequest=1563

Check failure on line 105 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/internal/RestletClientStreamTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8ziRd1Q3-BF5g5v&open=AaFrU8ziRd1Q3-BF5g5v&pullRequest=1563

Check warning on line 142 in org.restlet.ext.grpc/src/main/java/org/restlet/ext/grpc/internal/RestletTransportFactory.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant qualifier.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8zpRd1Q3-BF5g55&open=AaFrU8zpRd1Q3-BF5g55&pullRequest=1563

Check warning on line 71 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcHttpValidationTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8wqRd1Q3-BF5g5t&open=AaFrU8wqRd1Q3-BF5g5t&pullRequest=1563

Check warning on line 500 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g5y&open=AaFrU8znRd1Q3-BF5g5y&pullRequest=1563

Check warning on line 508 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/GrpcClientAdapterTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code of the lambda to have only one invocation possibly throwing a runtime exception.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8znRd1Q3-BF5g5z&open=AaFrU8znRd1Q3-BF5g5z&pullRequest=1563

Check failure on line 108 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/internal/RestletClientStreamTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8ziRd1Q3-BF5g5w&open=AaFrU8ziRd1Q3-BF5g5w&pullRequest=1563

Check warning on line 20 in org.restlet.ext.grpc/src/test/java/org/restlet/ext/grpc/internal/RestletClientStreamTestCase.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'io.grpc.internal.StreamListener.MessageProducer'.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8ziRd1Q3-BF5g5x&open=AaFrU8ziRd1Q3-BF5g5x&pullRequest=1563

Check warning on line 88 in org.restlet.ext.grpc/src/main/java/org/restlet/ext/grpc/internal/RestletClientStream.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Constructor has 11 parameters, which is greater than 7 authorized.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8zsRd1Q3-BF5g57&open=AaFrU8zsRd1Q3-BF5g57&pullRequest=1563

Check failure on line 139 in org.restlet.ext.grpc/src/main/java/org/restlet/ext/grpc/internal/RestletClientStream.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 17 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=restlet_restlet-framework-java&issues=AaFrU8zsRd1Q3-BF5g59&open=AaFrU8zsRd1Q3-BF5g59&pullRequest=1563