Skip to content

Add note about self-redirects - #237

Merged
drubery merged 1 commit into
mainfrom
push-xmpwmywvwuzl
Nov 21, 2025
Merged

drubery merged 1 commit into
mainfrom
push-xmpwmywvwuzl

Conversation

@drubery

@drubery drubery commented Nov 21, 2025

Copy link
Copy Markdown
Collaborator

It wasn't clear in #236 that this was possible. But it's a nice technique, so mention it in a non-normative section.

@drubery
drubery requested a review from thefrog-gh November 21, 2025 00:34

@thefrog-gh thefrog-gh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clever strategy, thanks for documenting it.

Comment thread spec.bs Outdated

Sites wishing to do a point-in-time challenge on a particular request can
accomplish this by triggering a redirect that also expires a bound cookie. For
example, returning an HTTP 307 to the same endpoint will force a DBSC refresh

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
example, returning an HTTP 307 to the same endpoint will force a DBSC refresh
example, returning an HTTP 307 to the same endpoint with a Set-Cookie header expiring the cookie will force a DBSC refresh

Comment thread spec.bs Outdated
preventing an easy cross-site leak of login state.

Sites wishing to do a point-in-time challenge on a particular request can
accomplish this by triggering a redirect that also expires a bound cookie. For

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
accomplish this by triggering a redirect that also expires a bound cookie. For
This results in a refresh followed by retrying the original request.

It wasn't clear in #236 that
this was possible. But it's a nice technique, so mention it in a
non-normative section.
@drubery
drubery merged commit e77532c into main Nov 21, 2025
2 checks passed
@drubery
drubery deleted the push-xmpwmywvwuzl branch November 21, 2025 17:33
github-actions Bot added a commit that referenced this pull request Nov 21, 2025
SHA: e77532c
Reason: push, by drubery

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

2 participants