Skip to content

fix(cli): set default User-Agent header#2564

Merged
benfrank241 merged 1 commit into
vectorize-io:mainfrom
koriyoshi2041:fix/cli-user-agent-20260705
Jul 7, 2026
Merged

fix(cli): set default User-Agent header#2564
benfrank241 merged 1 commit into
vectorize-io:mainfrom
koriyoshi2041:fix/cli-user-agent-20260705

Conversation

@koriyoshi2041

Copy link
Copy Markdown
Contributor

Summary

  • set a default User-Agent for the CLI reqwest client
  • keep the existing bearer auth default header behavior
  • add unit coverage for CLI default headers with and without an API key

Fixes #2538.

Tests

  • rustfmt --edition 2021 --check hindsight-cli/src/api.rs
  • cargo test --manifest-path hindsight-cli/Cargo.toml api::tests::default_headers -- --nocapture
  • cargo test --manifest-path hindsight-cli/Cargo.toml api::tests -- --nocapture

Note: full cargo fmt --manifest-path hindsight-cli/Cargo.toml --check is currently blocked by pre-existing formatting diffs in other CLI files outside this patch.

@benfrank241 benfrank241 merged commit e839c65 into vectorize-io:main Jul 7, 2026
84 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No user-agent in CLI triggers WAF rules

2 participants