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.

Sign up for GitHub

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

Jump to bottom

Show a notice, if the configured caching method is no longer available #305

Closed
stklcode opened this issue Aug 6, 2024 · 0 comments · Fixed by #304
Closed

Show a notice, if the configured caching method is no longer available #305

stklcode opened this issue Aug 6, 2024 · 0 comments · Fixed by #304
Assignees
@stklcode
Labels
enhancement
Milestone
2.4.0

Comments

@stklcode
Copy link
Contributor

stklcode commented Aug 6, 2024

We currently check for availability of the backend, e.g. if if a compatible PHP extension is available for external object caches or permalink structure is active for HDD caching. If this is not the case, we silently fall back to Database Caching which is always available.

We should at least add an admin notification to attract attention, because the plugin configuration does not fit the server environment which leads to potentially unintended behavior.
stklcode added a commit that referenced this issue Aug 6, 2024
@stklcode
introduce NOOP backend and admin notice for unavailable backend (#305)
28d7b4b 
We should not silently ignore misconfiguration and fall back to
DB caching. We now check for availability and disable caching
via a "no operation" backend. We can also add a notice for APC
now which is no longer supported.
@stklcode stklcode linked a pull request Aug 6, 2024 that will close this issue
remove APC support #304
Merged
stklcode added a commit that referenced this issue Aug 6, 2024
@stklcode
introduce NOOP backend and admin notice for unavailable backend (#305)
f45e3f9 
We should not silently ignore misconfiguration and fall back to
DB caching. We now check for availability and disable caching
via a "no operation" backend. We can also add a notice for APC
now which is no longer supported.
stklcode added a commit that referenced this issue Aug 6, 2024
@stklcode
introduce NOOP backend and admin notice for unavailable backend (#305)
70a629d 
We should not silently ignore misconfiguration and fall back to
DB caching. We now check for availability and disable caching
via a "no operation" backend. We can also add a notice for APC
now which is no longer supported.
@stklcode stklcode added this to the 2.4.0 milestone Sep 22, 2024
@stklcode stklcode self-assigned this Sep 22, 2024
stklcode added a commit that referenced this issue Sep 22, 2024
@stklcode
introduce NOOP backend and admin notice for unavailable backend (#305)
dd6de15 
We should not silently ignore misconfiguration and fall back to
DB caching. We now check for availability and disable caching
via a "no operation" backend. We can also add a notice for APC
now which is no longer supported.
@stklcode stklcode mentioned this issue Sep 22, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stklcode stklcode

Labels
enhancement
Projects
None yet
Milestone
2.4.0
Development

Successfully merging a pull request may close this issue.

remove APC support pluginkollektiv/cachify
1 participant
@stklcode