Skip to content

Commit

Permalink
Proton's cache-control time is set to 10 mins but docs lie
Browse files Browse the repository at this point in the history
Per the cache control response header emitted by RESTBase and the
yaml file, the cache-control response header for Proton's PDF is
set for 10 minutes instead of 5 minutes as the docs mentions.
  • Loading branch information
xSavitar committed Mar 15, 2023
1 parent 399e648 commit aff5fa6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion config.frontend.test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ default_project: &default_project
# Cache control for purged endpoints allowing short-term client caching
purged_cache_control_client_cache: test_purged_cache_control_with_client_caching
pdf:
# Cache PDF for 5 minutes since it's not purged
# Cache PDF for 10 minutes since it's not purged
cache_control: s-maxage=600, max-age=600
uri: https://proton-beta.wmflabs.org
transform:
Expand Down
2 changes: 1 addition & 1 deletion config.fullstack.test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ default_project: &default_project
# Cache control for purged endpoints allowing short-term client caching
purged_cache_control_client_cache: test_purged_cache_control_with_client_caching
pdf:
# Cache PDF for 5 minutes since it's not purged
# Cache PDF for 10 minutes since it's not purged
cache_control: s-maxage=600, max-age=600
uri: https://proton-beta.wmflabs.org
transform:
Expand Down

0 comments on commit aff5fa6

Please sign in to comment.