Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add initial nmap-service-probes crate #1059

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Conversation

XAMPPRocky
Copy link
Collaborator

It was mentioned that we should try to parse out pings from scanners separately, so that we can flag those packets separately from just "dropped packets". This PR adds the first step to doing that, which is adding support for the nmap-service-probes file format. This format is used by nmap for it to define its probes, and they have a large file that represents all of the probes they use. This gives us easy programmatic access to those payloads. I intend to followup with a PR that uses this crate to make a code generator so we can generate a set of types and constants we can use in the quilkin crate, but I wanted to make the crate PR separate since it is completely independent.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@XAMPPRocky XAMPPRocky merged commit 9bd548e into main Jan 13, 2025
12 checks passed
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: b0baf8fb-dadd-46dd-8567-fcdcf9fa075c

The following development images have been built, and will exist for the next 30 days:

To build this version:

git fetch [email protected]:googleforgames/quilkin.git pull/1059/head:pr_1059 && git checkout pr_1059
cargo build

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

Successfully merging this pull request may close these issues.

3 participants