Re: [ServiceWorker] Cache should sanitize the stored Request in put/add/addAll (#664)

@jungkees , I was talking @annevk on IRC and he pointed out we only want the sanitize step in .add() and .addAll().  So just removing the sanitize in the other methods I think removes the clone() calls I was concerned about.

Also, can we put the sanitize step in .add()/.addAll() after we check the request method?  That way if someone accidentally passes a PUT Request we won't drain the body stream when we error.

Thanks!

---
Reply to this email directly or view it on GitHub:
https://github.com/slightlyoff/ServiceWorker/issues/664#issuecomment-88918203

Received on Thursday, 2 April 2015 14:06:54 UTC