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

Add Bearer (JWT) authentication #233

Merged
merged 16 commits into from
Jun 5, 2024
Merged

Add Bearer (JWT) authentication #233

merged 16 commits into from
Jun 5, 2024

Conversation

lafrech
Copy link
Contributor

@lafrech lafrech commented Apr 23, 2024

Also rework Basic authentication.

Copy link

codecov bot commented Apr 23, 2024

Codecov Report

Attention: Patch coverage is 96.61017% with 4 lines in your changes missing coverage. Please review.

Project coverage is 99.36%. Comparing base (5d1a45e) to head (9891cf1).

Files Patch % Lines
src/bemserver_api/extensions/authentication.py 94.80% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #233      +/-   ##
==========================================
- Coverage   99.39%   99.36%   -0.03%     
==========================================
  Files         191      192       +1     
  Lines        4622     4721      +99     
  Branches     1613     1630      +17     
==========================================
+ Hits         4594     4691      +97     
- Misses         14       15       +1     
- Partials       14       15       +1     

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

@lafrech lafrech merged commit 22a18c1 into main Jun 5, 2024
6 of 8 checks passed
@lafrech lafrech deleted the jwt branch June 5, 2024 15:54
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