don't allow caching with secure (https) connections. #292

Closed
wants to merge 1 commit into
from

2 participants

@mattt

I'm not sure this is the right behavior. HTTPS does not inherently imply anything about the content being transported over the wire.

It's more common on browsers, where an entire site (including its assets) are served over https, so as to be secure, while also avoiding the dreaded mixed protocol warning in the browser. As for APIs, HTTPS is a bit more exotic, but when it is used, it doesn't necessarily signify anything that should change cacheing behavior. In cases where this is a problem, I think between the cacheing callback block property on operations, and setting the right cache policy, it's simple enough to override this behavior as necessary.

@steipete

Fair enough. I didn't spend to much time thinking about it, just saw that in the Apple docs and figured it might be a good idea.

@mattt

No worries. Thanks for keeping AFN in your mind as you read through that :)

@mattt mattt closed this Apr 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment