Skip to content

fix: Improve fernet keys loading#290

Merged
gtema merged 1 commit intomainfrom
fernet
Oct 10, 2025
Merged

fix: Improve fernet keys loading#290
gtema merged 1 commit intomainfrom
fernet

Conversation

@gtema
Copy link
Collaborator

@gtema gtema commented Oct 10, 2025

  • trim trailing line breaks, as those are making keys invalid.
  • raise explicit exception when no usable keys has been found.
  • log warning when a key file was not considered usable.

Irrelevant: make config.auth section mandatory

Since without specifying auth.methods the whole thing is not able to
work make the section non-default.

- trim trailing line breaks, as those are making keys invalid.
- raise explicit exception when no usable keys has been found.
- log warning when a key file was not considered usable.

Irrelevant: make config.auth section mandatory

Since without specifying auth.methods the whole thing is not able to
work make the section non-default.
@github-actions
Copy link

🐰 Bencher Report

Branchfernet
Testbedubuntu-latest
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
microseconds (µs)
(Result Δ%)
Upper Boundary
microseconds (µs)
(Limit %)
fernet token/project📈 view plot
🚷 view threshold
1.15 µs
(-50.49%)Baseline: 2.33 µs
9.69 µs
(11.91%)
🐰 View full continuous benchmarking report in Bencher

@gtema gtema merged commit 5848063 into main Oct 10, 2025
10 checks passed
@gtema gtema deleted the fernet branch November 3, 2025 11:09
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

Comments