Skip to content

[feat] Feat/caching auth#96

Merged
St4NNi merged 8 commits intofeat/version2.0reworkfrom
feat/caching-auth
Jul 28, 2023
Merged

[feat] Feat/caching auth#96
St4NNi merged 8 commits intofeat/version2.0reworkfrom
feat/caching-auth

Conversation

@St4NNi
Copy link
Copy Markdown
Member

@St4NNi St4NNi commented Jul 28, 2023

A basic / crude implementation of caching / authorization including path traversal.

  • Check contexts of requests
  • Check user permissions based on cached values
  • Decode Tokens based on pubkeys

@St4NNi St4NNi requested a review from das-Abroxas July 28, 2023 06:26
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 28, 2023

Codecov Report

Patch coverage: 31.33% and project coverage change: +3.48% 🎉

Comparison is base (67e1d59) 14.47% compared to head (b3377d2) 17.96%.
Report is 3 commits behind head on feat/version2.0rework.

Additional details and impacted files
@@                    Coverage Diff                    @@
##           feat/version2.0rework      #96      +/-   ##
=========================================================
+ Coverage                  14.47%   17.96%   +3.48%     
=========================================================
  Files                         35       36       +1     
  Lines                       2721     3022     +301     
=========================================================
+ Hits                         394      543     +149     
- Misses                      2327     2479     +152     
Files Changed Coverage Δ
src/auth/permission_handler.rs 0.00% <0.00%> (ø)
src/auth/structs.rs 0.00% <0.00%> (ø)
src/auth/token_handler.rs 0.00% <0.00%> (ø)
src/database/dsls/mod.rs 0.00% <0.00%> (ø)
src/database/dsls/pub_key_dsl.rs 0.00% <0.00%> (ø)
src/grpc/collections.rs 0.00% <0.00%> (ø)
src/grpc/datasets.rs 0.00% <0.00%> (ø)
src/grpc/notification.rs 0.00% <0.00%> (ø)
src/grpc/object.rs 0.00% <0.00%> (ø)
src/grpc/projects.rs 0.00% <0.00%> (ø)
... and 8 more

... and 2 files with indirect coverage changes

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

Copy link
Copy Markdown
Contributor

@das-Abroxas das-Abroxas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing obviously wrong. Let's go 🚀

@St4NNi St4NNi merged commit 34deb7a into feat/version2.0rework Jul 28, 2023
@St4NNi St4NNi deleted the feat/caching-auth branch July 28, 2023 07:58
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.

2 participants