entry.request.bodySize is incorrect for POST
Reported by
paulgr...@gmail.com,
Jan 9
|
||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/71.0.3578.98 Safari/537.36 Steps to reproduce the problem: 1. open dev tools 2. attempt sign in at https://accounts.google.com 3. export network traffic as HAR-with-content 4. open HAR in text editor What is the expected behavior? `entry.request.bodySize` should be... "Size of the request body (POST data payload) in bytes. Set to -1 if the info is not available." (http://www.softwareishard.com/blog/har-12-spec/#request) Maybe there needs to be some thought as to how to represent the request bodySize, as for a response, the `entry.response.bodySize` is represented as -1 with a new field `entry.response._transferredSize` representing the actual size of the response data sent over-the-wire. Tools like HAR Viewer cannot represent request bodies accurately without this information. What went wrong? `entry.request.bodySize` is 0, instead of the size of the request body. Did this work before? N/A Chrome version: 71.0.3578.98 Channel: stable OS Version: 10.0 Flash Version:
,
Jan 9
,
Jan 9
,
Jan 11
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/485cefab3e375f14041c7ad8f6b72cebe0294ead commit 485cefab3e375f14041c7ad8f6b72cebe0294ead Author: Joey Arhar <jarhar@chromium.org> Date: Fri Jan 11 01:38:16 2019 [DevTools] Correct entry.request.bodySize in HAR exports Bug: 920166 Change-Id: Ic37602ed429c46ad132780bb17fb8a3a2a4e594a Reviewed-on: https://chromium-review.googlesource.com/c/1405514 Commit-Queue: Joey Arhar <jarhar@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#621855} [modify] https://crrev.com/485cefab3e375f14041c7ad8f6b72cebe0294ead/third_party/blink/renderer/devtools/front_end/sdk/HARLog.js [add] https://crrev.com/485cefab3e375f14041c7ad8f6b72cebe0294ead/third_party/blink/web_tests/http/tests/devtools/network/har-post-expected.txt [add] https://crrev.com/485cefab3e375f14041c7ad8f6b72cebe0294ead/third_party/blink/web_tests/http/tests/devtools/network/har-post.js
,
Jan 11
Thanks for the find! |
||||
►
Sign in to add a comment |
||||
Comment 1 by phanindra.mandapaka@chromium.org
, Jan 9