Improve measure of the number of bytes by adding cost of HTTP headers. about carbonalyser HOT 4 CLOSED

What about size of underlying header they seem to be not taken in account in the compute:
20 bytes for ip
20 bytes for tcp
some bytes for TLS if thereis

from carbonalyser.

  let length = 0;
  for(var a = 0; a < requestDetails.responseHeaders.length; a =a +1) {
    var obj = requestDetails.responseHeaders[a];
    length += (obj.name + ": " + obj.value).length;
  }

from carbonalyser.

A first version is implemented by 04885a4.

For instance on https://www.w3.org/Icons/w3c_home here the total number of bytes for the request:
We get 9826 bytes with the old version.

With the new version we got 10891 (total) on an expected value of 10985 (2 * (20 + 20) + (headers http) + (content http).

Measure of http part size is down via devtools. We do not get as much as theorically expected because of others elements of the http header that we cannot handle (http error status response for instance).

In a more advanced version we may count headers sent by the client (at the webRequest.onSendHeaders). This amount should be attributed to sites visited also and not to clients because clients are slave of the website.

from carbonalyser.

This is next part is done at 708c1a8.

For instances on https://www.w3.org/Icons/w3c_home there is two request.

we measure (bytes) 472 and 409 bytes versus 507 and 441 in devtools as previous this is due to other http protocal elements.

from carbonalyser.