Skip to content

Merge stable/8.x changes forward.#225

Merged
szmyd merged 4 commits into
masterfrom
stable/v8.x
May 29, 2024
Merged

Merge stable/8.x changes forward.#225
szmyd merged 4 commits into
masterfrom
stable/v8.x

Conversation

@szmyd

@szmyd szmyd commented Feb 12, 2024

Copy link
Copy Markdown
Collaborator

@shosseinimotlagh Can you resolve these conflicts so we don't loose changes in the 8.x branch? I'm not sure what should be merged into 11 and what should be un-touched.

@codecov-commenter

codecov-commenter commented Feb 23, 2024

Copy link
Copy Markdown

Codecov Report

Merging #225 (aa54f72) into master (1e1f648) will increase coverage by 0.11%.
Report is 168 commits behind head on master.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
+ Coverage   63.74%   63.86%   +0.11%     
==========================================
  Files          72       72              
  Lines        4273     4273              
  Branches      527      527              
==========================================
+ Hits         2724     2729       +5     
+ Misses       1320     1315       -5     
  Partials      229      229              
Components Coverage Δ
AuthManager 88.38% <ø> (ø)
Cache 26.17% <ø> (+0.75%) ⬆️
FDS 68.32% <ø> (+0.09%) ⬆️
FileWatcher 56.25% <ø> (ø)
Flip 67.42% <ø> (ø)
gRPC 77.12% <ø> (ø)
Logging 28.66% <ø> (ø)
Metrics 80.20% <ø> (ø)
Options 100.00% <ø> (ø)
Setting 56.68% <ø> (ø)
StatusObject 73.83% <ø> (ø)
Utility 83.44% <ø> (ø)
Version 95.83% <ø> (ø)

* Add issued_at leeway and not_before leeway to jwt verifier. Remove LRU cache for tokens and always verify them using jwt library

* add unit tests

* download token 5 minutes before expiry

* use LRU cache for valid tokens

* request new token during the half life of the existing token

---------

Co-authored-by: Ravi Nagarjun Akella <raakella1@$HOSTNAME>
@szmyd
szmyd merged commit dd253e3 into master May 29, 2024
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.

4 participants