-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
A qlog file formated with json-seq #82
Comments
Hey @nixwl, qvis should just support the JSON-SEQ format as well, so there should be no need to convert to JSON (as such, there are also no existing tools for this (though it should be easy enough to write one, it's quite simple)). Can you share an example file and the implementation you're using maybe? If you're actually getting errors loading the file into qvis, which errors? |
hi, I can import JSON-SEQ format qlog file to website,but I found that: This qlog file record client download file form a QUIC file server(the QUIC server uses TQUIC) and client qlog file is here: qlog_c.txt I don't know if my qlog file has any problems, but the connection may be established, and file download function had well worked |
I take that issue to TQUIC developer few days before. And they make a scipt to covert JSON-SEQ to JSON, So I try it recently. This is the result:(converted qlog file is here: qlog_c.qlog.json) |
So it turns out that the JSON-SEQ file you use isn't actually proper JSON-SEQ as per the qlog spec :) Since they don't do that, qvis can't properly parse the file and shows an error in the JavaScript console (I agree it's confusing that it does show a green popup in the webpage UI, pretending that it did load correctly ;)). So, the solution here is to ask the TQUIC maintainers to either output proper JSON-SEQ as per the spec, or for you to use their script to convert to JSON, which indeed does seem to work as expected. (I will keep this issue open to remind myself to one day properly show a red popup with an error message if this error occurs again, rather than people having to open the JavaScript console in the developer tools). |
The text was updated successfully, but these errors were encountered: