Code Monkey home page Code Monkey logo

Comments (3)

sbansal7 avatar sbansal7 commented on May 27, 2024

Here are the diffs for a failing test:

diff --git a/runtime/fieldmask_test.go b/runtime/fieldmask_test.go
index 1af66ab4..3db815ef 100644
--- a/runtime/fieldmask_test.go
+++ b/runtime/fieldmask_test.go
@@ -28,6 +28,13 @@ func TestFieldMaskFromRequestBody(t *testing.T) {
                        name:     "empty",
                        expected: newFieldMask(),
                },
+               {
+                       name: "EmptyMessage",
+
+                       msg:      &examplepb.ABitOfEverything{},
+                       input:    `{"oneof_empty": {}}`,
+                       expected: newFieldMask("oneof_empty"),
+               },
                {
                        name: "simple",

from grpc-gateway.

johanbrandhorst avatar johanbrandhorst commented on May 27, 2024

Thanks for your issue! I'm a little confused - you have a field where the presence of an empty message is significant? Surely an empty value isn't set or unset, it's just empty? What is the semantic meaning of a "set" empty value?

That said, if you want to contribute a fix for this, I'm all for it. I just don't really get it.

from grpc-gateway.

sbansal7 avatar sbansal7 commented on May 27, 2024

Thanks for response. Empty messages are useful as default choice in oneof. Similar to how it is defined here: https://github.com/grpc-ecosystem/grpc-gateway/blob/main/examples/internal/proto/examplepb/a_bit_of_everything.proto#L295

Here is another example. Lets say, you want the user to select a site and also give the user an option to explicitly select "all_sites", you could define a oneof like this:

oneof site_option {
    google.protobuf.Empty all_sites = 1;
    string site = 2;
  }

Now, when someone sends all_sites, I was hoping that I will get all_sites set in fieldMask, but it is not set.

from grpc-gateway.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.