tls-parser icon indicating copy to clipboard operation
tls-parser copied to clipboard

combine_dtls_fragments helper

Open algesten opened this issue 1 year ago • 3 comments

~# REQUIRES #47 TO LAND FIRST #~

This PR builds upon #47. The aim is to provide a correct helper that combines DTLS fragments into a full message. The test case illustrates how the parser + combine helper are expected to be used together.

algesten avatar Feb 03 '24 14:02 algesten

Will rebase when/if #47 lands.

algesten avatar Feb 03 '24 14:02 algesten

I believe this is ready to merge now.

algesten avatar Apr 23 '24 10:04 algesten

I believe this is ready to merge now.

Hi, After review I need more input on the choices you made (see above comments). The most important questions are on the Ordered type, others can be solved separately. Thanks!

chifflier avatar Apr 23 '24 14:04 chifflier