-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
This PR will trigger a minor release when merged. |
Codecov Report
@@ Coverage Diff @@
## master #131 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 11
Lines 293 293
=========================================
Hits 293 293
Continue to review full report at Codecov.
|
🎉 This PR is included in version 1.8.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
@@ -46,7 +46,7 @@ async function extract(url, params, log = console) { | |||
statusCode: 200, | |||
headers: { | |||
'Content-Type': 'application/json', | |||
'Cache-Control': 'max-age=600', | |||
'cache-control': 'no-store, private, must-revalidate', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems overly aggressive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
discussed with @davidnuescheler that we should only cache in fastly. as it is invoked by the content-proxy, which is requested by fastly, this is the most transparent way to handle it.
Adobe I/O Runtime respects the `no-cache` directive in the `Cache-Control` *request* header, which will allow us to do a partial revert of adobe/helix-data-embed#131 and keep some caching as the default see adobe/helix-data-embed#153
fixes #90