Skip to content

docs: protected sub-resources answer 401 instead of redirecting - #6102

Open
mshabarov wants to merge 1 commit into
mainfrom
docs/unauthenticated-subresource-401
Open

mshabarov wants to merge 1 commit into
mainfrom
docs/unauthenticated-subresource-401

Conversation

@mshabarov

Copy link
Copy Markdown
Contributor

Documents vaadin/flow#25593

security · flow/security · Spring Security applications with a
configured login view

An unauthenticated request for a protected stylesheet, script, image or
font used to be redirected to the login view. The redirect couldn't
work and ended in an ERR_TOO_MANY_REDIRECTS error that pointed at the
login view instead of the blocked resource. Such requests now get a
plain 401 Unauthorized. The configurer reference described neither
the old behavior nor the new one.

Risks:

  • ✅ Docs only. One paragraph added, no moved anchors.

  • Added a paragraph to the Login View Configuration section of the
    VaadinSecurityConfigurer reference, with a since badge.

  • Said which requests get the 401, which keep the redirect, and that
    the Sec-Fetch-Dest header decides, so a browser that doesn't send
    it is treated as asking for a page.

🤖 Generated with Claude Code

A stylesheet, script, image or font requested without a session used to
be redirected to the login view, which ended in a redirect loop that
hid the resource that was denied.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@peholmst peholmst added the target/v25.3 Automatically cherry-pick to the v25.3 branch label Sep 21, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Preview Deployment

This PR has been deployed for preview.

URL: https://docs-preview-pr-6102.fly.dev

Changed pages

Added content is highlighted in green; removed content is marked in red on each page.

Built from be9182b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

target/v25.3 Automatically cherry-pick to the v25.3 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants