Skip to content

Revert "Render unauthenticated session expire redirect as meta tag"#7273

Merged
aduth merged 3 commits intomainfrom
revert-7249-aduth-meta-expire-redirect
Nov 2, 2022
Merged

Revert "Render unauthenticated session expire redirect as meta tag"#7273
aduth merged 3 commits intomainfrom
revert-7249-aduth-meta-expire-redirect

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Nov 1, 2022

Reverts #7249

Related Slack conversation: https://gsa-tts.slack.com/archives/C0NGESUN5/p1667331820015699

The changes in #7249, while functionally equivalent to what had existed previously, introduce new errors flagged by accessibility testing tools related to http-equiv="refresh". Future iterations should explore how to allow a user to extend the timeout, while remaining compatible with technical requirements for CSRF token expiration of 15 minutes.

(Draft while I undo some of the diff to keep some desirable changes like template comment corrections and spec removal) Edit: Edited in 3438c5e.

@aduth aduth marked this pull request as ready for review November 1, 2022 20:19
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@aduth aduth merged commit bd47ab1 into main Nov 2, 2022
@aduth aduth deleted the revert-7249-aduth-meta-expire-redirect branch November 2, 2022 12:12
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