-
-
Notifications
You must be signed in to change notification settings - Fork 402
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
feat: security requirements override #4018
base: main
Are you sure you want to change the base?
feat: security requirements override #4018
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4018 +/- ##
=======================================
Coverage 98.35% 98.35%
=======================================
Files 347 347
Lines 15783 15786 +3
Branches 1745 1746 +1
=======================================
+ Hits 15523 15526 +3
Misses 124 124
Partials 136 136 ☔ View full report in Codecov by Sentry. |
Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/4018 |
…fran/litestar into fix-allow-empty-security-requirements
@tofran There seem to be some test failures. Can you address those? |
Looks like linting is failing in the upstream. It started failing by updating the branch based on main.
Maybe I can make a separate PR to addrress this. |
Closes #4016
This started as a fix to the issue above, but after thorough consideration this needs a feature which I called
security_override
.With
security_override
you can override OpenAPI's security requirements passed viasecurity
in any layer of your application (app, router, route).Refer to the included docstrings for more information.