Added do_dissect() to TLSRecord

alexmgr commented 9 years ago

Check this approach, it's simpler and should work in all cases: . For records such as TLSRecord()/TLSHandshake()/TLSClientHello()/TLSRecord() will have the same behavior as before. . Stacked records are handled correctly by SSL(), including payloads, and extensions *. Stacked records can be created through TLS.from_records([stacked_record1, stacked_record2])

Fixed SSL() do_dissect
Added support to SSL() for stacked records
Records are truncated at the length reported in the header
Made extension headers conditonal
Added unittests

tintinweb commented 9 years ago

will rebase this PR tomorrow. feel free to merge it if you're faster :) good work :+1:

buildhive commented 9 years ago

tintin » scapy-ssl_tls #6 SUCCESS This pull request looks good (what's this?)

alexmgr commented 9 years ago

My bad. Thanks for fixing, probably a rebase conflict resolution mistake.

tintinweb / scapy-ssl_tls

Added do_dissect() to TLSRecord #16