Patch for ... #5

Open
wants to merge 1 commit into
from

Projects

None yet

4 participants

@elliotttf
Member

I don't have a problem with this in principle, maybe @davidstrauss has some thoughts though.

Have you tried this with nginx or another non-apache webserver to make sure it still behaves correctly?

You might consider and update hook to force the path_cache to clear when this change goes out so there's not wasted space on unreachable paths.

@davidstrauss
Member

@elliotttf

Have you tried this with nginx or another non-apache webserver to make sure it still behaves correctly?

This should work fine on any web server.

You might consider and update hook to force the path_cache to clear when this change goes out so there's not wasted space on unreachable paths.

I personally wouldn't bother. Most caches use a combination of expiration and LRU that make keeping around harmless stale items better than explicitly spending time in PHP to clear them.

@fuzzy76
fuzzy76 commented Mar 27, 2014

The URL to the original issue is dead :-/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment