only cache GET requests #5

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants

leedo commented May 7, 2012

While doing load tests we noticed that ab was hitting this condition and not being cached. Apparently it is including some POST data in the GET request (maybe just some white space or newline?)

It seems to me that batcache should only be caching GET requests anyways. So this commit just checks the request method.

Member

skeltoac commented May 11, 2012

Could you post the full HTTP requests that are hitting this condition?

Member

skeltoac commented May 11, 2012

Also if you could inspect the variables in that test, it would help in narrowing down the cause of this.

Contributor

vnsavage commented Oct 19, 2015

Closing this request as no further data was provided and it's not clear how to reproduce this.

vnsavage closed this Oct 19, 2015

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