-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Fixes #6938] Basic auth middleware #6939
[Fixes #6938] Basic auth middleware #6939
Conversation
Merge from upsteram
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @allyoucanmap on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added. |
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @allyoucanmap on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added. |
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @allyoucanmap on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added. |
Two failing tests due to temporarly unavailable Remote Service. Not related to the current code. |
* allow Basic authenticated requests in LOCKDOWN mode * fix to avoid circular import * flake8 check fix * added tests Co-authored-by: Giovanni Allegri <[email protected]> Co-authored-by: allyoucanmap <[email protected]> Co-authored-by: biegan <[email protected]> (cherry picked from commit 4c8e824)
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.