Skip to content
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

Forcibly delete rules cache on flush #28

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

trepmal
Copy link
Contributor

@trepmal trepmal commented Dec 4, 2019

We should ensure that the rewrite rules cache is emptied when flushing the rules

@GaryJones GaryJones added this to the 1.3.0 milestone May 10, 2021
@GaryJones
Copy link
Contributor

Was this fixing a bug? Or just a belt and braces approach to reduce the likelihood of a bug?

@trepmal
Copy link
Contributor Author

trepmal commented May 10, 2021

There's a brief history in this Slack thread.

Essentially, there had been several instances where a forced cache delete seemed to be required after flushing the rules. However, that shouldn't be required as the relevant cache should be flushed when the option is changed.

I've also not been aware of this issue recurring in the last several months, so something else may have been at play when this was happening.

@GaryJones
Copy link
Contributor

Thanks for the reference.

Since it's not been happening recently, let's close this out, but it's good to have this as a potential solution if it starts happening again.

@davidsword
Copy link

davidsword commented Feb 17, 2023

This has been popping up again. If performing a flush, I think it's very reasonable to flush rewrite_rules from the options group as well, given the non-zero chance of rewrite_rules ending up there. A flush should flush everything.

@davidsword davidsword reopened this Feb 17, 2023
@GaryJones GaryJones added this to the 1.3.2 milestone May 16, 2024
@GaryJones GaryJones merged commit 565223e into main May 16, 2024
@GaryJones GaryJones deleted the trepmal-force-cache-flush branch May 16, 2024 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants