idunno.Authentication icon indicating copy to clipboard operation
idunno.Authentication copied to clipboard

#146 Update Basic Auth handler and Options to enable overriding SuppressWWWAuthenticateHeader

Open joursler-Sunset opened this issue 1 year ago • 0 comments

Update Basic Auth handler and Options to enable overriding SuppressWWWAuthenticateHeader configuration setting based on the root of the path.

Checklists for Pull requests

About pull request itself:

  • [X] I am submitting a pull request :)
  • [X ] I have already created an issue and referred to it in the commit message.
  • [X ] My submission does one logical thing only (one bugfix, one new feature). If want to supply multiple logical changes, I will submit multiple pull requests.
  • [X ] I have read and understood CONTRIBUTING guide

Commits:

  • [X ] My commits are logical, easily readable, with concise comments.

Licensing:

  • [X ] I am the author of submission or have been authorized by submission copyright holder to issue this pull request.

Branching:

  • [X ] My submission is based on master branch.
  • [X ] My submission is compatible with latest master branch updates (no conflicts, I did a rebase if it was necessary).
  • [X ] The name of the branch I want to merge upstream is not 'master' (except for only the most trivial fixes, like typos and such).
  • [X ] My branch name is feature/my-new-not-for-productionfeature (for new features).
  • [ ] My branch name is bugfix/i-fixed-your-crap-code (for bugfixes).

Pull request description

Add the necessary code to override the SuppressWWWAuthenticateHeader based on a root path to support libraries that have their own UI and need to send the WWW-Authenticate header while the rest of the application does not.

joursler-Sunset avatar Dec 18 '24 18:12 joursler-Sunset