Skip to content

Add optional api_key Basic auth for mutating CLI requests - #36

Merged
tsauvajon merged 2 commits into
masterfrom
enable-auth
Aug 24, 2026
Merged

tsauvajon merged 2 commits into
masterfrom
enable-auth

Conversation

@tsauvajon

Copy link
Copy Markdown
Owner

No description provided.

- Config gains an optional api_key (skipped when absent, so legacy
  configs deserialize unchanged); when set, POST/PUT requests carry
  Authorization: Basic base64(api_key) and GET requests never do
- Bump version to 2.1.0
- Front-end: explicit top-level sign-in link to ./_goto/login so a
  proxy's interactive login flow can redirect the whole page
- README: document the credential format (username:app-password for
  Authentik forward-auth edges) and 0600 file permissions
- Tests: base64 RFC 4648 vectors, request-builder header assertions,
  httpmock wire checks incl. proving GET leaks no credentials;
  coverage is now 100% lines/regions/functions (llvm-cov), clippy clean
@codecov

codecov Bot commented Aug 24, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.95918% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 97.02%. Comparing base (e9d746f) to head (cb839aa).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/cli/main.rs 97.95% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage   95.58%   97.02%   +1.44%     
==========================================
  Files           2        2              
  Lines         272      303      +31     
==========================================
+ Hits          260      294      +34     
+ Misses         12        9       -3     
Flag Coverage Δ
rust 97.02% <97.95%> (+1.44%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tsauvajon
tsauvajon merged commit 29d893d into master Aug 24, 2026
6 checks passed
@tsauvajon
tsauvajon deleted the enable-auth branch August 24, 2026 19:32
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.

1 participant