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

basicauth directive has no way to supply exceptions #13

Closed
Lemmmy opened this issue Mar 4, 2021 · 0 comments
Closed

basicauth directive has no way to supply exceptions #13

Lemmmy opened this issue Mar 4, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@Lemmmy
Copy link
Member

Lemmmy commented Mar 4, 2021

Unless I'm missing some obvious 'trick', the basicauth directive does not have an easy way to supply exceptions (e.g. via an except option); the only ways I can see are to list all resources except the one you want an exception for (inconvenient and error-prone), or to set up an additional site block that bypasses the auth plugin directly. I think that, similar to the proxy directive, basicauth should have an except option.

The use case I have in mind for this is a PWA protected behind basicauth. The user-agent requests to get /manifest.json on each page load, however, Chrome (and possibly other browsers) does not send Authorization headers for this particular request, meaning the console gets spammed with 401 errors, and fails Lighthouse checks.

@Lemmmy Lemmmy added the enhancement New feature or request label Mar 4, 2021
@1lann 1lann closed this as completed in 52c171f Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant