- From: Ben Kelly <notifications@github.com>
- Date: Thu, 02 Apr 2015 07:06:24 -0700
- To: slightlyoff/ServiceWorker <ServiceWorker@noreply.github.com>
Received on Thursday, 2 April 2015 14:06:54 UTC
@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