HarViewer does not give parsing error with the modification.#22
HarViewer does not give parsing error with the modification.#22prongs wants to merge 2 commits intoandrewf:masterfrom
Conversation
|
Thanks, but pcap2har can't unconditionally add a jsonp call, especially with a hard-coded string. The original HAR works fine in the original HAR Viewer (when you disable validation, due to a long-standing minor bug w.r.t timing). A command-line option would be fine. |
|
So you want me to add a command line option to check whether or not to add these lines? |
|
Um, yes, if you think it's important. I would prefer the jsonp function name itself to be configurable too. I hate to throw a bunch of extra requirements at you, but this is a weird use case and, well, I'm picky. Actually, I was just able to manually upload your original non-jsonp file to pcapperf without any problem, after downloading it to my hdd. Might it be a bug in pcapperf or the HAR viewer? |
harviewer gives error when a har generated by pcap2har(example) is fed to it(example, see error in console).
After the modification, new har file and the harviewer output.