Use full URL (url.host + url.request_uri) as cache key

[vmanivchuk]

No description provided.

[olleolleolle]

The "full key" form disambiguates, which is better.

(Keeping the old form around for compatibility - is it necessary, in your opinion? I think it may not be necessary.)

@technoweenie How did this work before? Did site_a.com/path and site-b.com/path return the same cached content? (That's my understanding from this diff.) If yes, then let's apply this immediately. ❤️ 🌴

[vmanivchuk]

Keeping the old form for compatibility is necessary because there can be issue with key length. And yes, site_a.com/path and site-b.com/path return the same cached content.

[olleolleolle]

Would it be acceptable to switch over to this new behavior in an upcoming major version? Update: Ah, keys can be too long for one's cache implementation, yeeees. Thanks for clarifying.

[vmanivchuk]

Yes, of course.

[iMacTia]

Wanted to include this in last 0.1x release as it's a pretty good point!