New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed #33060 -- Added BaseCache.make_and_validate_key() hook. #14802
Conversation
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.
A couple comments.
42a14ad
to
97e570a
Compare
Merged the 3rd commit in 68b8eda. |
576923f
to
670d5ff
Compare
@ngnpope Thanks 👍 I rebased and pushed edits to tests. |
670d5ff
to
1f6bfb9
Compare
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.
OK, this looks neat. Thanks all!
Commit message niggle — Not sure about ...which will be stored
here:
Refs #33060 -- Ensured cache backends validate keys which will be stored.
The validate_key() function should be called after make_key() to ensure that the validation is performed on the key that will actually be stored in the cache. Co-authored-by: Mariusz Felisiak <felisiak.mariusz@gmail.com>
This helper function reduces the amount of duplicated code and makes it easier to ensure that we always validate the keys.
@carltongibson Thanks for the review 👍 Suggestions applied. |
1f6bfb9
to
42dfa97
Compare
Thanks for polishing this off @felixxm 🦸🏻 |
ticket-33060
Following on from #14437 (comment).