Skip to content

Support JSON Name in pb field names #1214

@renevo

Description

@renevo

Feature request checklist

  • There are no issues that match the desired change
  • The change is large enough it can't be addressed with a simple Pull Request
  • If this is a bug, please file a Bug Report.

Change

Allow for the protobuf field name mapping to use the JSONName instead of the field names.

Example

person.firstName vs. person.first_name

Alternatives considered

I have temporarily forked this repo to support this, it was a pretty simple change, however I am not sure of a good way to make this configurable either through a global var or a build tag for a PR.

master...Blizzard:cel-go:blizzard

Ideally I don't want to maintain this for for such a small thing. The reason why this is needed in our use case is that we allow for CEL queries to filter data in a graphql API. With the graphql schema being based on the JSON of the data, it is way less confusing to our users than having to do the mental dance of the above example.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions