Skip to content

Conversation

@itambek-godaddy
Copy link
Contributor

@itambek-godaddy itambek-godaddy commented Sep 15, 2021

Summary

Release branch for v5.10.10

Individual PRs should branch off from master so they can be tested without updating the FW namespace from individual plugins (out of convenience).

PRs

The filter is applied in the API base class because the request class itself has no access to the plugin instance, which is required to scope the filter to the plugin.
The new filters around cache lifetime and whether a request is cacheable are more suitable for disabling or debugging request caching.
This will allow correctly caching GET requests as well as prevent issues where a POST request with the exact same body is made to 2 different endpoints, and a cached response from the first endpoint is returned.
Copy link
Contributor

@ChaseWiseman ChaseWiseman left a comment

Choose a reason for hiding this comment

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

Change PRs already reviewed and approved, FW version changes look good.

@ragulka ragulka merged commit b7fb94e into master Sep 23, 2021
@unfulvio unfulvio deleted the release/5.10.10 branch January 10, 2022 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants