Contribute to Scapy: a master list of good first issues

[gpotter2]

Hi & welcome to Scapy's github ! This page lists issues that you can try to fix if you want to start contributing to Scapy.

Disclaimer

The BEST way to contribute to Scapy is to play around with it, to find and fix broken stuff by yourself, i.e. Before you attempt to fix one of those pesky issues, just HAVE FUN with Scapy !😺

By using Scapy in your work, experiments, hacks, courses... you'll often end up finding more bugs/issues/unclear stuff and contributing way more than by reading through this list of things, which is far more valuable to us.

Need inspiration? Have a look at awesome-scapy to see what people do with it !

Still here?

This list includes wishes and things added by the maintainers based on the issues that we get, but also issues marked with TODO or XXX that already exist in Scapy's code base (layers). If you want to contribute to the project you might just take care one of the bugs.

You can also look for issues and pull requests in the old Bitbucket repository. If you find something interesting there (there shouldn't be anything left, but sometimes lost people add issues there), please do not update it on Bitbucket, open an issue here and mention (link) the old issue and/or pull request.

To get a bug, create an issue with the description of the bug and refer to this issue (just mention #399 in the text) so we can update it.

Please read How to contribute before starting to work!

Note: all of those issues are updated regularly but might sometimes fall out of date. The links are static so the part you need to change might move on master (look for it). If the issue has been fixed and we missed it, feel free to ping us

Available:

Easy - Recommended for new users - Hacktoberfest

• Any of the issues tagged with "good first issue"

• Participate in project Hinty: [Hinty] Add type hints to Scapy #2158. This is easy but requires a lot of hands to help 😄. You'll just have to add static typing to a file or layer you've selected.

• Add documentation for dispatch_hook based on this StackOverflow post. You can add this documentation to scapy/doc/scapy/advanced_usage.rst for instance.

• Add hashret()/answers() regression tests to ICMPv6ND_RA in regression.uts as said
  

  scapy/test/regression.uts

  Line 1692 in 45c5930

  # TODO: Add answers()/Hashret() tests ( think about Cisco routers

  . This is a good way to discover what our unit test infrastructure looks like (it's a bit outdated though very complete.. we're trying to switch to pytest....)

Medium - More advanced features or wishes

• create a snap package for Scapy (snap [Feature Request] #2153) (apparently that's something Linux users want)

• Implement TCP_server like TCP_client (Feature request: TCP_server #2083), except it should handle multiple streams ! (Implement Transmission Control Block? This also leads to understanding how Scapy automatons work)

• Improve in6_getAddrType: add support for Unique local Address and add unit tests. (this function appears to be unused, try pinging guedou about that)
  

  scapy/scapy/utils6.py

  Line 191 in c6e1e6c

  # TODO : integrate Unique Local Addresses

• Add more tests in regression.uts for IA_PD Prefix Option in
  

  scapy/test/regression.uts

  Line 3264 in 45c5930

  #TODO : finish me

• Make the FlagField a bit cleaner thanks to the new capabilities of FlagsField in isakmp.py (This should be easy enough, try help(FlagsField) to see what this field is about)
  

  scapy/scapy/layers/isakmp.py

  Line 185 in cce88a9

  FlagsField("flags",0, 8, ["encryption","commit","auth_only","res3","res4","res5","res6","res7"]), # XXX use a Flag field

• Don't remove Teredo's route when refreshing in route6.py
  

  scapy/scapy/route6.py

  Line 41 in cce88a9

  # TODO : At the moment, resync will drop existing Teredo routes

  (I'm honnestly not sure what this is about. Ask us :p)

Hard - Build entire modules or refactor complex stuff

• add SNMPv3 support SNMPv3 packet support #1132

• add DTLS support Missing support for DTLS layer #1931 (This is very cool, but not easy. We do have full TLS support though so it should be feasible to re-use almost everything)

• Update the p0f support p0f rewritten #1923 (This sounds pretty cumbersome...)

• Remove the "trick" from carp.py
  

  scapy/scapy/contrib/carp.py

  Line 47 in cce88a9

  # XXX: this is a dirty hack. it needs to pack version and type into a single 8bit field

  (You very likely need to get back to the RFC to understand what the heck is going on here. At least this should be rather simple)

• Try to call Enum_metaclass from ASN1_Class_metaclass or at least try to reduce the amount of duplicated code:
  

  scapy/scapy/asn1/asn1.py

  Line 113 in a6e4ab2

  def __new__(cls, name, bases, dct): # XXX factorise a bit with Enum_metaclass.__new__()

• Update the OPC UA support [WIP] OPC UA #1569 (This sounds extremely hard, considering how big the PR is. Don't dive into this unless you actually need OPC UA)

• Add support to addresses other than MAC in DUID_LLT in dhcp6.py
  

  scapy/scapy/layers/dhcp6.py

  Line 204 in cce88a9

  # XXX We only support Ethernet addresses at the moment. _LLAddrField

• Add support for authdata in OSPF_Hdr in ospf.py
  

  scapy/scapy/contrib/ospf.py

  Line 68 in f912538

  # TODO: Support authdata (which is appended to the packets as if it were padding)

• Guess the next hop properly when multiple IPv6ExtHdrDestOpt extensions are used on inet6.py
  

  scapy/scapy/layers/inet6.py

  Line 379 in bdcbba5

  nh = self.payload.nh # XXX what if another extension follows ?

  (You might want to check the RFC: should we use the one from the last option? or the first one? (current))

• Check that Multicast scopes match when performing answers in [inet6.py]
  

  scapy/scapy/layers/inet6.py

  Line 407 in bdcbba5

  # XXX test mcast scope matching ?

• Write X.400 format in X509.py. (this is ASN.1. Not the easiest thing to begin with)
  

  scapy/scapy/layers/x509.py

  Line 199 in cce88a9

  #XXX write me

HELP: the RFC of the the X.400 format is on rfc5280, at the bottom of page 117.

Unknown difficulty

• Fix a bug with ICMPv6's hashret in inet6.py
  

  scapy/scapy/layers/inet6.py

  Line 424 in 35f97ef

  # TODO: big bug with ICMPv6 error messages as the destination of IPerror6

  (this looks like a bug but it's not crystal clear... 😕)