Skip to content

Fix docs typo X-Requested-By -> X-Requested-With#18123

Merged
rwinch merged 1 commit intospring-projects:mainfrom
martinboulais:patch-1
Jan 8, 2026
Merged

Fix docs typo X-Requested-By -> X-Requested-With#18123
rwinch merged 1 commit intospring-projects:mainfrom
martinboulais:patch-1

Conversation

@martinboulais
Copy link
Copy Markdown
Contributor

Simple documentation typo fix

The documentation states that setting the header X-Requested-By will remove the WWW-Authenticate header from the response.

However, after testing this and reading the library code it looks like the header to set should be X-Requested-With (X-Requested-By is mentioned nowhere except in this documentation file), so I propose this simple PR to fix this.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Nov 2, 2025
@rwinch rwinch changed the title Fix typo in HTTP Basic Auth Provider documentation Fix docs typo X-Requested-By -> X-Requested-With Jan 8, 2026
@rwinch rwinch self-assigned this Jan 8, 2026
@rwinch rwinch added in: docs An issue in Documentation or samples type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 8, 2026
@rwinch rwinch added this to the 7.0.3 milestone Jan 8, 2026
The documentation states that setting the header `X-Requested-By` will remove the `WWW-Authenticate` header from the response.
However, after testing this and reading the library code it looks like the header to set is `X-Requested-With` (X-Requested-By is mentioned nowhere except in this documentation file), so I propose this simple PR to fix this.

Signed-off-by: Martin Boulais <31805063+martinboulais@users.noreply.github.com>
@rwinch rwinch enabled auto-merge (rebase) January 8, 2026 19:47
@rwinch
Copy link
Copy Markdown
Member

rwinch commented Jan 8, 2026

Thanks for the pull request. I've rebased it and it will be merged as soon as the build passes 😄

@rwinch rwinch merged commit 1d8ea63 into spring-projects:main Jan 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: docs An issue in Documentation or samples type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants