Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: app auth cache keys missing app_id #95

Merged
merged 4 commits into from
Mar 20, 2024

Add app_id to JWT app cache keys

89a4a7f
Select commit
Loading
Failed to load commit list.
Merged

Bug: app auth cache keys missing app_id #95

Add app_id to JWT app cache keys
89a4a7f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Mar 19, 2024 in 1s

35.21% (-0.01%) compared to 57ead1b

View this Pull Request on Codecov

35.21% (-0.01%) compared to 57ead1b

Details

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 35.21%. Comparing base (57ead1b) to head (89a4a7f).

❗ Current head 89a4a7f differs from pull request most recent head 2364ca5. Consider uploading reports for the commit 2364ca5 to get more accurate results

Files Patch % Lines
githubkit/auth/app.py 14.28% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
- Coverage   35.21%   35.21%   -0.01%     
==========================================
  Files        2430     2430              
  Lines      124901   124903       +2     
==========================================
  Hits        43986    43986              
- Misses      80915    80917       +2     

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