search

webmetrics / browsermob-proxy

NOTICE: this project has been forked and is being maintained at https://github.com/lightbody/browsermob-proxy
https://github.com/lightbody/browsermob-proxy
Apache License 2.0
232 stars 763 forks source link

Simonz change to browsermob proxy handler #106

Closed roydekleijn closed 11 years ago

roydekleijn commented 11 years ago

It did look promising, but tests are taking twice as long and they finish with errors.

So I closed it for now, maybe we have to do more extensive testing on it.

lightbody commented 11 years ago

Weird. Why would tests take longer because of this? Seems unlikely.

roydekleijn commented 11 years ago

I tried the same test a couple of times, with and without this change. There was a difference in time and bmp threw some errors when this code was enabled