cloudflare
lib, qlog: Track and Log Unknown Transport Parameters
Track (and make available to qlog for logging) transport parameters and their values that this implementation does not specifically recognize.
@LPardue Let me know what you think about the direction of this PR. If you think that it is on the right track, I will gladly do whatever you say so that it meets the high standards of the code that you have already written in this library.
In re: https://github.com/quicwg/qlog/issues/436 and (potentially) https://github.com/quicwg/qlog/issues/176.
Sorry it took me so long, @LPardue ! If this version looks okay, I will start in on determining how to handle FFI!
LGTM thanks @hawkinsw .
I'd like @ghedo to take a final look over before merging it though
Hello @LPardue! I noticed that we missed an opportunity to add a member function to test whether an unknown transport parameter's id is in the reserved space. I have updated the PR! Let me know what you think!
(Also, I rebased to make it easier to review).
@ghedo Thank you for the review! I just talked to @LPardue and I let him know that I will address these bits of feedback tonight! Thank you, again!
Added a couple additional minor tweaks (e.g. renamed safe_push -> push) and merged, thanks @hawkinsw!
