Skip to content

fix: cli error message and logging #34

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

Merged
leoparente merged 2 commits into from
Jul 14, 2025
Merged

fix: cli error message and logging #34

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d44722b
fix: cli error message and logging
leoparente Jul 14, 2025
256e646
use log
leoparente Jul 14, 2025
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
5 changes: 4 additions & 1 deletion cmd/diode-replay-dryrun/main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ func (f *fileList) Set(s string) error {
func main() {
var files fileList
flag.Var(&files, "file", "Dry-run JSON file to ingest (may be repeated)")
target := flag.String("target", "", "gRPC target of the Diode server, e.g. grpc://localhost:8080/diodet")
target := flag.String("target", "", "gRPC target of the Diode server, e.g. grpc://localhost:8080/diode")
app := flag.String("app-name", "", "Application name used when ingesting the dry-run messages")
version := flag.String("app-version", "", "Application version used when ingesting the dry-run messages")
clientID := flag.String("client-id", "", "OAuth2 client ID. Defaults to the DIODE_CLIENT_ID environment variable if not provided")
Expand All @@ -37,8 +37,11 @@ func main() {
*clientSecret = os.Getenv("DIODE_CLIENT_SECRET")
}

log.SetFlags(0)

if len(files) == 0 || *target == "" || *app == "" || *version == "" {
flag.Usage()
log.Println("Error: the following arguments are required: -target, -app-name, -app-version, -file")
os.Exit(1)
}

Expand Down
Loading