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

arp: add tests #1711

Closed
wants to merge 1 commit into from
Closed

arp: add tests #1711

wants to merge 1 commit into from

Conversation

glongo
Copy link
Contributor

@glongo glongo commented Mar 18, 2024

Ticket

If your pull request is related to a Suricata ticket, please provide
the full URL to the ticket here so this pull request can monitor
changes to the ticket status:

Redmine ticket:
https://redmine.openinfosecfoundation.org/issues/6827

@glongo
Copy link
Contributor Author

glongo commented Mar 18, 2024

Suricata PR: OISF/suricata#10659

@glongo
Copy link
Contributor Author

glongo commented Mar 20, 2024

Suricata PR: OISF/suricata#10675

@catenacyber catenacyber added the requires suricata pr Depends on a PR in Suricata label Mar 20, 2024
@glongo
Copy link
Contributor Author

glongo commented Mar 21, 2024

Suricata PR: OISF/suricata#10692

@glongo
Copy link
Contributor Author

glongo commented Mar 23, 2024

Suricata PR: OISF/suricata#10705

Copy link
Member

@jasonish jasonish left a comment

Choose a reason for hiding this comment

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

I'd like to see a test on the arp reply as well.

@glongo
Copy link
Contributor Author

glongo commented Apr 2, 2024

I'd like to see a test on the arp reply as well.

Done in #1739

@glongo glongo closed this Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires suricata pr Depends on a PR in Suricata
Development

Successfully merging this pull request may close these issues.

3 participants