Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Default URI Exclusion Patterns -- same as old version? #41

Closed
bridgeport opened this Issue · 5 comments

3 participants

@bridgeport

I recently purchased the Pro version. So far, at least in my testing, it's operating very well.

In the old version of Quick Cache (v111203), you had these default URI Exclusion Patterns:

wp-app
wp-signup
wp-register
wp-activate
wp-login
wp-admin
xmlrpc

The Pro version doesn't insert any stock defaults. Is the above still recommended as necessary exclusions? If so, are there any that are no longer valid and any additional ones that should be added to account for WordPress' various updates since way back when?

Thanks.

@jaswsinc
Owner

In the latest version of Quick Cache and Quick Cache Pro these can be supplied if you wish (that's fine). However, they are no longer necessary because these are automatically excluded by Quick Cache and Quick Cache Pro now; even if you omit them :-)

@bridgeport

Thanks for confirming this. If I omit "wp-login" from the URI exclusion list it looks like Quick Cache Pro does still cache that page, but not any of the others, such as "lostpassword" (I assume due to the query string).

Is that intended behavior? It seems caching "wp-login.php" is optimal as there's no actual danger of caching something user-specific (e.g., usernames) due to the POST request method being ignored by Quick Cache Pro.

@jaswsinc
Owner

@bridgeport You are correct, no real harm in this; but that's NOT the intended behavior. I'm having this fixed for the next official release. @raamdev Pull requests have been submitted to correct the bug reported here.

@bridgeport Thanks for identifying this!

@jaswsinc
Owner

Related commit. See: JasWSInc/Quick-Cache@7fb1062

@jaswsinc jaswsinc referenced this issue from a commit
Commit has since been removed from the repository and is no longer available.
@raamdev
Owner

This bug was fixed in Quick Cache v131224:

https://github.com/WebSharks/Quick-Cache/releases/tag/131224

@raamdev raamdev closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.