Skip to content

Comments

DNP3#4923

Open
nrathaus wants to merge 7 commits intosecdev:masterfrom
nrathaus:dnp3
Open

DNP3#4923
nrathaus wants to merge 7 commits intosecdev:masterfrom
nrathaus:dnp3

Conversation

@nrathaus
Copy link

Checklist:

  • If you are new to Scapy: I have checked CONTRIBUTING.md (esp. section submitting-pull-requests)
  • I squashed commits belonging together

Sorry I am not show how to do this, some guidance?

  • I added unit tests or explained why they are not relevant

Not sure if it is needed here or not - this is a new contrib protocol - should I add PCAP ? or something like s = raw(IP()...)?

  • I executed the regression tests (using tox)
  • If the PR is still not finished, please create a Draft Pull Request

--

This PR will introduce DNP3 support to scapy, it was done about 9 years ago: https://github.com/nrodofile/ScapyDNP3_lib but never integrated (unknown reason)

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.

1 participant