Skip to content

Add TCPCLv4 message definitions and tcp_reassemble interface - #5076

Draft
BrianSipos wants to merge 1 commit into
secdev:masterfrom
BrianSipos:add-tcpclv4
Draft

Add TCPCLv4 message definitions and tcp_reassemble interface#5076
BrianSipos wants to merge 1 commit into
secdev:masterfrom
BrianSipos:add-tcpclv4

Conversation

@BrianSipos

Copy link
Copy Markdown

Description

This is a port of earlier TCPCL packet definitions from dtn-demo-agent with more modern scapy-friendly patterns of dispatchers and with a tcp_reassemble() interface.

This is an alternative to part of #4824 with some simplified structures.
Test capture files are included for demonstration.

Comment thread scapy/contrib/tcpcl.py
return conf.raw_layer


class ContactV3(_BaseContact):

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it preferred that all public classes use a globally-unique prefix for user experience consistency?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If possible yeah, it allows to avoid conflicts. It's up to you to use TCPCL_Contact or TCPCLContact, or some other method :)

It also helps when auto-completing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants