fullstorydev / fullstorydev/grpcurl

Change -emit-defaults behaviour so that the flag isn't anymore needed to show default values (Proto3)

Open
#217 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
12.8k
Forks
580
Avg merge
1h 8m
Merged PRs (30d)
5

Description

Hi,
I encountered the same issue as described in #97.
After some debugging, I tried to call the ame service with BloomRPC GUI and default value was shown immediately.
Some research lead me to -emit-defaults option which solves the issue for me.
Although this is expected behaviour as explained in #97, it would be desirable to have -emit-defaults set by default and give the user the option to omit defaults if needed.
This is obviously how BloomRPC and other clients/client libraries work.

Or is there anything against this approach?

Thanks!

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the grpcurl command-line handling for the -emit-defaults flag and tracing how default values are emitted. Define the default behavior and the option for omitting defaults, then verify the CLI behavior against the Proto3 case described in the issue; no specific file or test is named.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
cli
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.