Skip to content

Commit 6ab3680

Browse files
authored
feat: Use main HTTP port for gRPC (#252)
Fixes #250 🦕
1 parent d6a4d97 commit 6ab3680

File tree

4 files changed

+8
-4
lines changed

4 files changed

+8
-4
lines changed

.github/workflows/run-tck.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ jobs:
5555
- name: Build with Maven, skipping tests
5656
run: mvn -B install -DskipTests
5757
- name: Start SUT
58-
run: SUT_GRPC_URL=http://localhost:9000 mvn -B quarkus:dev & #SUT_JSONRPC_URL already set
58+
run: SUT_GRPC_URL=${{ env.SUT_JSONRPC_URL }} mvn -B quarkus:dev & #SUT_JSONRPC_URL already set
5959
working-directory: tck
6060
- name: Wait for SUT to start
6161
run: |

reference/grpc/src/test/java/io/a2a/server/grpc/quarkus/QuarkusA2AGrpcTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,9 @@ protected String getTransportProtocol() {
2929

3030
@Override
3131
protected String getTransportUrl() {
32-
return "localhost:9001"; // gRPC server runs on port 9001
32+
// gRPC server runs on port 8081, which is the same port as the main web server.
33+
// This is from the setting on the
34+
return "localhost:8081";
3335
}
3436

3537
@Override
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
# Configure the gRPC server to listen on port 9001
2-
quarkus.grpc.server.port=9001
1+
# Use the new gRPC implementation which uses the main HTTP port
2+
quarkus.grpc.server.use-separate-server=false
33
quarkus.http.port=8081
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,3 @@
1+
# Use the new gRPC implementation which uses the main HTTP port
2+
quarkus.grpc.server.use-separate-server=false
13
%dev.quarkus.http.port=9999

0 commit comments

Comments
 (0)