#help
Title
# help
f
Faye Salwin
11/07/2023, 7:01 PMdoes anyone have any tips on someone who doesn't live in the java world might go about debugging why I can't serialize a message that I got in response to querying the actual server using grpcurl? Can I get the server to dump a readable version of the dsc?
Copy code
Exception in thread "grpc-default-executor-0" wiremock.com.google.protobuf.InvalidProtocolBufferException: Cannot find field: matchId in message genius_sports_replica.v1alpha.Match
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.mergeMessage(JsonFormat.java:1498)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.merge(JsonFormat.java:1458)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.parseFieldValue(JsonFormat.java:2018)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.mergeRepeatedField(JsonFormat.java:1733)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.mergeField(JsonFormat.java:1665)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.mergeMessage(JsonFormat.java:1500)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.merge(JsonFormat.java:1458)
at wiremock.com.google.protobuf.util.JsonFormat$ParserImpl.merge(JsonFormat.java:1340)
at wiremock.com.google.protobuf.util.JsonFormat$Parser.merge(JsonFormat.java:472)
at org.wiremock.grpc.internal.JsonMessageUtils.lambda$toMessage$1(JsonMessageUtils.java:36)
at com.github.tomakehurst.wiremock.common.Exceptions.uncheck(Exceptions.java:70)
at org.wiremock.grpc.internal.JsonMessageUtils.toMessage(JsonMessageUtils.java:36)
at org.wiremock.grpc.internal.UnaryServerCallHandler.lambda$invoke$0(UnaryServerCallHandler.java:92)
at com.github.tomakehurst.wiremock.http.AbstractRequestHandler.handle(AbstractRequestHandler.java:101)
at org.wiremock.grpc.internal.UnaryServerCallHandler.invoke(UnaryServerCallHandler.java:60)
at org.wiremock.grpc.internal.UnaryServerCallHandler.invoke(UnaryServerCallHandler.java:31)
at wiremock.grpc.io.grpc.stub.ServerCalls$UnaryServerCallHandler$UnaryServerCallListener.onHalfClose(ServerCalls.java:182)
at wiremock.grpc.io.grpc.internal.ServerCallImpl$ServerStreamListenerImpl.halfClosed(ServerCallImpl.java:351)
at wiremock.grpc.io.grpc.internal.ServerImpl$JumpToApplicationThreadServerStreamListener$1HalfClosed.runInContext(ServerImpl.java:860)
at wiremock.grpc.io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37)
at wiremock.grpc.io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:133)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)testing with the demo app, It's fine if I name fields foo_bar or fooBar (except in $. syntax where it really wants fooBar (field is defined in proto as foo_bar)
t
Tom
11/07/2023, 7:38 PMThe dsc is compiled directly from the .proto so you should be able to use that to inspect the structure
If you want to look at the JSON WireMock is seeing you can enable
--verbosef
Faye Salwin
11/08/2023, 5:41 AMSo, I've downloaded grpcui which can take a protoset (dsc) file and make requests like grpcurl but from a web ui. I loaded the protoset I'm loading into wiremock, I sent the request and I get back a response, no errors. This means it has correctly parsed the response message. The response data is json and matches the underscore version of the JSON. it reports no problems. When I try to make the same request, using this returned json as the body of the mapping response, I get an error that it cannot find the first field of the repeated message which causes a stack trace to be dumped by wiremock. I'm using v0.2 which we setup to work yesterday and I've confirmed that the naive example still works in wiremock.
I have just noticed, however:
Copy code
WARNING: The descriptor for message type ".genius_sports_replica.v1alpha.Match" cannot be found and a placeholder is created for itis it failing to match .genius_sports_replica and genius_sports_replica?
Copy code
package genius_sports_replica.v1alpha;
import "genius_sports_replica/v1alpha/competitor.proto";
import "genius_sports_replica/v1alpha/venue.proto";
import "google/protobuf/duration.proto";
import "google/protobuf/timestamp.proto";
message Match {
int32 match_id = 1;
protoc --include_imports --descriptor_set_out mock/wiremock/genius/grpc/api.dsc -I ../../genius-sports-replica/api/proto -I ../../googleapis -I ../../grpc-gateway $fileactually I'm seeing .genius_sport_replica all over the generated dsc
so the difference is, I did not create the proto from the directory containing the api. If I do, the file generated is different
OK, I merged all the .proto for that module into a single file
and now the only warnings I'm getting are about the google imports
so, I think you have a problem parsing the imports.
BOOM, with that change, I was able to return mock data.
and able to test it...
SERIOUSLY, thanks so much for this software
if we can get to the bottom of the dsc parsing problem (trim leading dots?) we'll be good I think
I was able to resolve the issues I was having using wiremock grpc extension by merging all the files for my module into a single file (avoiding imports for all except google)... obviously this isn't something everyone will reasonably be able to do, so I hope you can fix the imported objects - looks like maybe the leading dot is the issue and either adding to all, or removing from all might well solve the problem.
t
Tom
11/08/2023, 9:33 AMPlease could you share a project that replicates the issue you’ve described?
Agree it would be good to get to the bottom of this, even if it’s just to add a warning in the docs.
OK, confirmed this is an issue, fix in progress: https://github.com/wiremock/wiremock-grpc-extension/pull/13
🙌 1
f
Faye Salwin
11/08/2023, 5:58 PMConfirmed, I've rebuilt my docker containers with this new standalone jar 0.3.0 and it's working in my test with the original dsc
t
Tom
11/08/2023, 6:04 PMThanks for confirming!
7 Views