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

throw if auth header header not present/empty #9

Open
ryzokuken opened this issue Aug 4, 2021 · 1 comment
Open

throw if auth header header not present/empty #9

ryzokuken opened this issue Aug 4, 2021 · 1 comment

Comments

@ryzokuken
Copy link

IIUC, the middleware currently proceeds normally if auth header is not present or is empty.

There should be a way to require a valid auth and fail on those cases.

@ryzokuken
Copy link
Author

For now, I've hardcoded the other behavior in a fork (ryzokuken@5e0857e) and used it in my project as a stopgap measure. Perhaps we could add options like bail_without_header on the structs that could be set by the caller on initialization and check those like should_401_on_multiple_values()... maybe a configurable should_401_without_auth()? Would you accept a PR with that change?

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

No branches or pull requests

1 participant