search

backdrop / backdrop-issues

Issue tracker for Backdrop core.
144 stars 38 forks source link

[DX] .htaccess: Change the cryptic value of `A1209600` for `ExpiresDefault` to a human-readable one #6646

Open klonos opened 1 month ago

klonos commented 1 month ago

Current value is this:

  # Cache all files for 2 weeks after access (A).
  ExpiresDefault A1209600

Proposal to change that to this instead:

  # Cache all files for 2 weeks after access.
  ExpiresDefault "access plus 2 weeks"

See spec for details: https://httpd.apache.org/docs/current/mod/mod_expires.html

avpaderno commented 1 month ago

With ExpiresDefault "access plus 2 weeks", it is not even necessary to comment that line.

klonos commented 1 month ago

OK, here's a PR to get things going: https://github.com/backdrop/backdrop/pull/4827

I haven't completely removed the comment for now - lets wait for some feedback and adjust once others had a chance to chime in.

avpaderno commented 1 month ago

I was going to suggest adding a link to a page explaining the syntax used for that directive, but then I noticed that is not done for the other directives as well.

izmeez commented 1 month ago

I like this improvement.