220.127.116.11. Contributing to Suricata
This guide describes what steps to take if you want to contribute a patch or patchset to Suricata.
Essentially, these are:
Agree to and sign our Contribution Agreement
Communicate early, and use the preferred channels
Follow our Coding Style
Use our Documentation Style
Stick to our commit guidelines
Add version numbers to your Pull Requests
Incorporate Feedback into new PRs
[Work merged] Wrap up!
The rest of this document will cover those in detail.
Before contributing, please review and sign our Contribution Agreement.
18.104.22.168.1. Communication is Key!
To clarify questions, discuss or suggest new features, talk about bugs and optimizations, and/or ask for help, it is important to communicate.
These are our main channels:
22.214.171.124.2. Claim (or open) a ticket
For features and bugs we need tickets. Tickets help us keep track of the work done, indicate when changes need backports etc.
They are also important if you would like to see your new feature officially added to our tool: the ticket documents your ideas so we can analyze how do they fit in our plans for Suricata, and, if the feature is accepted, we can properly track progress etc.
If you want to add new functionalities (e.g. a new application layer protocol), please ask us first whether we see that being merged into Suricata or not. This helps both sides understand how the new feature will fit in our roadmap, and prevents wasting time and motivation with contributions that we may not accept. Therefore, before starting any code related to a new feature, do request comments from the team about it.
For really trivial fixes or cleanups we won't need that.
Once work on the issue has been agreed upon:
Assign the ticket to yourself. For this, you will need to have the "developer" role. You can ask for that directly on the ticket you want to claim or mention that you are interested in working on ticket number on our Developer's channel on Discord.
If a ticket is already assigned to someone, please reach out on the ticket or ask the person first.
You can reach out to other community members via Suricata's Discord server.
If you submit a new feature that is not part of Suricata's core functionalities, it will have the community supported status. This means we would expect some commitment from you, or the organization who is sponsoring your work, before we could approve the new feature, as the Suricata development team is pretty lean (and many times overworked).
This means we expect that:
the new contribution comes with a set of Suricata-verify tests (and possibly unit tests, where those apply), before we can approve it;
proof of compatibility with existing keywords/features is provided, when the contribution is for replacing an existing feature;
you would maintain the feature once it is approved - or some other community member would do that, in case you cannot.
Regardless of contribution size or complexity, we expect that you respect our guidelines and processes. We appreciate community contributors: Suricata wouldn't be what it is without them; and the value of our tool and community also comes from how seriously we take all this, so we ask that our contributors do the same!
126.96.36.199.3.1. What does "community supported" and "supporting a feature" mean?
If a feature is community supported, the Suricata team will try to spend minimal time on it - to be able to focus on the core functionalities. If for any reason you're not willing or able to commit to supporting a feature, please indicate this.
The team and/or community members can then consider offering help. It is best to indicate this prior to doing the actual work, because we will reject features if no one steps up.
It is also important to note that community supported features will be disabled by default, and if it brings in new dependencies (libraries or Rust crates) those will also be optional and disabled by default.
Supporting a feature means to actually maintain it:
keeping it up to date
offering end-user support via forum and/or Discord chat
188.8.131.52.4. Stale tickets policy
We understand that people's availability and interested to volunteer their time to our project may change. Therefore, to prevent tickets going stale (not worked on), and issues going unsolved for a long time, we have a policy to unclaim tickets if there are no contribution updates within 6 months.
If you claim a ticket and later on find out that you won't be able to work on it, it is also appreciated if you inform that to us in the ticket and unclaim it, so everyone knows that work is still open and waiting to be done.
184.108.40.206.5. What branch to work on
There are 2 or 3 active branches:
master-x.x.x (e.g. master-6.x.y)
The former is the stable branch. The latter the development branch.
The stable branch should only be worked on for important bug fixes. Those are mainly expected from more experienced contributors.
Development of new features or large scale redesign is done in the development
branch. New development and new contributors should work with
in very special cases - which should and would be discussed with us first.
If in doubt, please reach out to us via Redmine, Discord or forum.
220.127.116.11.6. Create your own branch
It's useful to create descriptive branch names. You're working on ticket 123 to improve GeoIP? Name your branch "geoip-feature-123-v1". The "-v1" addition is for feedback. When incorporating feedback you will have to create a new branch for each pull request. So, when you address the first feedback, you will work in "geoip-feature-123-v2" and so on.
For more details check: Creating a branch to do your changes
18.104.22.168.7. Coding Style
We have a Coding Style that must be followed.
22.214.171.124.8. Documentation Style
For documenting code, please follow Rust documentation and/or Doxygen guidelines, according to what your contribution is using (Rust or C).
If you are writing or updating documentation pages, please:
wrap up lines at 79 (80 at most) characters;
when adding diagrams or images, we prefer alternatives that can be generated automatically, if possible;
bear in mind that our documentation is published on Read the Docs and can also be built to pdf, so it is important that it looks good in such formats.
126.96.36.199.9. Commit History matters
Please consider our Commit guidelines before submitting your PR.
188.8.131.52.10. Send a Pull Request
The pull request is used to request inclusion of your patches into the main repository. Before it is merged, it will be reviewed and pushed through a QA process.
Please consider our Pull Requests Criteria when submitting.
We have 'GitHub-CI' integration enabled. This means some automated build check, suricata-verity and unit tests are performed on the pull request. Generally, this is ready after a few minutes. If the test fails, the pull request won't be considered. So please, when you submit something, keep an eye on the checks, and address any failures - if you do not understand what they are, it is fine to ask about them on the failing PR itself.
Before merge, we also perform other integration tests in our private QA-lab. If those fail, we may request further changes, even if the GitHub-CI has passed.
You'll likely get some feedback. Even our most experienced devs do, so don't feel bad about it.
After discussing what needs to be changed (usually on the PR itself), it's time to go back to "Create your own branch" and do it all again. This process can iterate quite a few times, as the contribution is refined.
184.108.40.206.12. Wrapping up
220.127.116.11.12.1. Merged! Cleanup
Congrats! Your change has been merged into the main repository. Many thanks!
We strongly suggest cleaning up: delete your related branches, both locally and on GitHub - this helps you in keeping things organized when you want to make new contributions.
18.104.22.168.12.2. Update ticket
You can now put the URL of the merged pull request in the Redmine ticket. Next, mark the ticket as "Closed" or "Resolved".
Well done! You are all set now.