Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adjust setCacheControl max-age to 5 #1707

Closed
wants to merge 1 commit into from
Closed

Adjust setCacheControl max-age to 5 #1707

wants to merge 1 commit into from

Conversation

alectrocute
Copy link
Contributor

Possibly fixes #1705.

If anybody has a better proposed fix, please do let me know and feel free to close this PR.

Also includes a formatting thing for security-handler.ts that was bugging me.

Thanks!

@SleeplessOne1917
Copy link
Member

The Clear-Site-Data header could be worth adding to login and logout requests.

@alectrocute
Copy link
Contributor Author

Closing in favor of #1708.

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

@dessalines dessalines Awaiting requested review from dessalines dessalines is a code owner

@SleeplessOne1917 SleeplessOne1917 Awaiting requested review from SleeplessOne1917 SleeplessOne1917 is a code owner

@jsit jsit Awaiting requested review from jsit

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refreshing page after logging in, appears to log you out.
2 participants
@alectrocute @SleeplessOne1917