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

HTTPBearer token is set, Auth button not shown on /api/docs #758

Merged
merged 20 commits into from
Oct 24, 2024

Conversation

Igoranze
Copy link
Contributor

@Igoranze Igoranze commented Oct 10, 2024

Part of #670

This PR adds a Depends and extracts the token from the request in order to allow for HTTPBearer requests.
Part of the missing HTTPBearer request was that the /api/docs auth button was not shown.

@Igoranze Igoranze changed the title WIP: HTTPBearer token set HTTPBearer token is set, Auth button not shown on /api/docs Oct 14, 2024
@Igoranze Igoranze closed this Oct 14, 2024
@Igoranze Igoranze reopened this Oct 14, 2024
Copy link

codecov bot commented Oct 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.76%. Comparing base (d5f46ba) to head (d97d6ac).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #758   +/-   ##
=======================================
  Coverage   83.76%   83.76%           
=======================================
  Files         190      190           
  Lines        9447     9450    +3     
  Branches      924      924           
=======================================
+ Hits         7913     7916    +3     
  Misses       1283     1283           
  Partials      251      251           

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

pyproject.toml Outdated Show resolved Hide resolved
Copy link
Contributor

@Mark90 Mark90 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@torkashvand torkashvand left a comment

Choose a reason for hiding this comment

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

LGTM!

@Igoranze Igoranze merged commit 6230904 into main Oct 24, 2024
10 checks passed
@Igoranze Igoranze deleted the auth-b-shown branch October 24, 2024 14: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.

4 participants