google.api.http options in multi-line format not supported

[grpc-ecosystem]

google.api.http options in multi-line format not supported

[yugui]

Thank you for letting me know that. But it looks to be an issue in protoc or googleapis.
Let me confirm to the upstream.

Although the extension google.api.http in MethodOptions is an optional field, protoc emits as if it is a repeated field for the syntax (2). I am not sure what is the expected behavior.

Some possibilities come to my mind.

1. protoc should have combined those lines into one extension if the two variations of syntax mean the same thing?
2. goprotobuf should have done it?
3. google.api.http should have been repeated field and grpc-gateway should have combine the repeated values into one?

Here is the dump of the code generation request.

...
    name: "EchoBody"
    input_type: ".gengo.grpc.gateway.example.SimpleMessage"
    output_type: ".gengo.grpc.gateway.example.SimpleMessage"
    options: <
      /* 38 unknown bytes */
      72295728: "\"\x15/v1/example/echo_body"
      72295728: ":\x01*"
    >

[sjincho]

Hi Yuki,

I found: protocolbuffers/protobuf#59

So seems more like a protoc issue, unfortunately.

2015-06-01 22:10 GMT+09:00 Yuki Yugui Sonoda [email protected]:

Thank you for letting me know that. But it looks to be an issue in protoc
or googleapis.
Let me confirm to the upstream.

Although the extension google.api.http in MethodOptions is an optional
field, protoc emits as if it is a repeated field for the syntax (2). I am
not sure what is the expected behavior.

Some possibilities come to my mind.

1. protoc should have combined those lines into one extension if the
   two variations of syntax mean the same thing?
2. goprotobuf should have done it?
3. google.api.http should have been repeated field and grpc-gateway
   should have combine the repeated values into one?

Here is the dump of the code generation request.

...
name: "EchoBody"
input_type: ".gengo.grpc.gateway.example.SimpleMessage"
output_type: ".gengo.grpc.gateway.example.SimpleMessage"
options: <
/* 38 unknown bytes /
72295728: ""\x15/v1/example/echo_body"
72295728: ":\x01"
>

—
Reply to this email directly or view it on GitHub
#16 (comment).