Escaping and better .htaccess hdd sample code #208
Conversation
timse201
changed the title
|
Hi @timse201, I tried to verify that the changes fix the issue with the cached post request, but I still get the cached version for post requests with the changed |
|
The PR does not fix #200, I will open a PR for that later today. I am not sure about the other changes, is escaping needed here, does that have any advantages over leaving the |
|
| RewriteCond %{REQUEST_URI} !^/(wp-admin|wp-content/cache)/.* | ||
| RewriteCond %{HTTP_COOKIE} !(wp-postpass|wordpress_logged_in|comment_author)_ | ||
| RewriteCond %{QUERY_STRING} ^$ | ||
| RewriteCond %{REQUEST_URI} !^\/(wp\-admin|wp\-content\/cache)\/.* |
There was a problem hiding this comment.
Is there any reason to escape the characters here?
/ is not used as a delimiter in this context and - is no reserved character outside character groups 🤔
|
Closing this PR in favor of #302 and other refactoring around that area. Real issues should be all resolved by now. Feel free to reopen this in case we missed something. |
closes #157