Skip to content

perf: Set session.cache_limiter at runtime to avoid clients caching static assets served by PHP - #46320

Merged
nickvergessen merged 1 commit into
masterfrom
perf/php-session-cache-limiter
Jul 10, 2024
Merged

perf: Set session.cache_limiter at runtime to avoid clients caching static assets served by PHP#46320
nickvergessen merged 1 commit into
masterfrom
perf/php-session-cache-limiter

Conversation

@juliusknorr

Copy link
Copy Markdown
Member

By default there is a Pragma: no-cache header set due to the default
value no-cache of session.cache-limiter, which will cause Chrome and
iOS to not cache even with a different Cache-Control header set on the
response.

Signed-off-by: Julius Härtl jus@bitgrid.net

  • Resolves: #

Summary

TODO

  • ...

Checklist

@juliusknorr
juliusknorrforce-pushed the perf/php-session-cache-limiter branch from 0d98e08 to 9c76acbCompareJuly 5, 2024 12:42
@juliusknorr
juliusknorr requested review from a team, SystemKeeper and nickvergessenJuly 5, 2024 12:47
@juliusknorr
juliusknorrforce-pushed the perf/php-session-cache-limiter branch from 9c76acb to 189cd15CompareJuly 5, 2024 13:58

@SystemKeeperSystemKeeper left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested and works 👍

…tatic assets served by PHP
By default there is a Pragma: no-cache header set due to the default
value `no-cache` of session.cache-limiter, which will cause Chrome and
iOS to not cache even with a different Cache-Control header set on the
response.
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr
juliusknorrforce-pushed the perf/php-session-cache-limiter branch from 189cd15 to 38bee2cCompareJuly 8, 2024 20:30
@SystemKeeper

Copy link
Copy Markdown
Contributor

Seems we can't convince cypress, but to me it seems not related??
Can we get this in to backport it for the 29/28 RC tomorrow?

@nickvergessen
nickvergessen merged commit e31f474 into masterJul 10, 2024
@nickvergessen
nickvergessen deleted the perf/php-session-cache-limiter branch July 10, 2024 09:52
@nickvergessen

Copy link
Copy Markdown
Member

/backport to stable29

@nickvergessen

Copy link
Copy Markdown
Member

/backport to stable28

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

Labels

3. to reviewWaiting for reviewsperformance 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@juliusknorr@SystemKeeper@nickvergessen