Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use json camel case when GetUseJSONNamesForFields is enabled #985

Merged
merged 10 commits into from
Aug 9, 2019
Merged

Use json camel case when GetUseJSONNamesForFields is enabled #985

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7ebf898
Use json name when GetUseJSONNamesForFields is enabled
xin-au Aug 8, 2019
5b00482
Optimzed an if statement
xin-au Aug 8, 2019
1e7abbf
Merge two if statements
xin-au Aug 8, 2019
ada726e
Convert parameters to lower camel case in url
xin-au Aug 8, 2019
dbe7dd8
Revert services.go file
xin-au Aug 9, 2019
14aad74
Fix an unit test
xin-au Aug 9, 2019
71e0de7
Update bazel file
xin-au Aug 9, 2019
697a950
Add an unit test for testing json camel case
xin-au Aug 9, 2019
20403f1
Add an unit test for testing code without enabling json camel case
xin-au Aug 9, 2019
b9ec55f
Add more corner cases
xin-au Aug 9, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 11 additions & 5 deletions protoc-gen-grpc-gateway/descriptor/services.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ func (r *Registry) loadServices(file *File) error {
for _, sd := range file.GetService() {
glog.V(2).Infof("Registering %s", sd.GetName())
svc := &Service{
File: file,
File: file,
ServiceDescriptorProto: sd,
}
for _, md := range sd.GetMethod() {
Expand Down Expand Up @@ -258,10 +258,16 @@ func (r *Registry) newResponse(meth *Method, path string) (*Body, error) {

// lookupField looks up a field named "name" within "msg".
// It returns nil if no such field found.
func lookupField(msg *Message, name string) *Field {
func (r *Registry) lookupField(msg *Message, name string) *Field {
for _, f := range msg.Fields {
if f.GetName() == name {
return f
if r.GetUseJSONNamesForFields() {
if f.GetJsonName() == name {
return f
}
} else {
if f.GetName() == name {
return f
}
}
}
return nil
Expand Down Expand Up @@ -291,7 +297,7 @@ func (r *Registry) resolveFieldPath(msg *Message, path string, isPathParam bool)
}

glog.V(2).Infof("Lookup %s in %s", c, msg.FQMN())
f := lookupField(msg, c)
f := r.lookupField(msg, c)
if f == nil {
return nil, fmt.Errorf("no field %q found in %s", path, root.GetName())
}
Expand Down