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

Output alert applayer v3 #1465

Closed
wants to merge 4 commits into from

Conversation

catenacyber
Copy link
Collaborator

@catenacyber catenacyber commented Nov 16, 2023

@victorjulien
Copy link
Member

should be rebased to include edef3ad

@catenacyber catenacyber force-pushed the output-alert-applayer-v3 branch from 873b16d to f589e81 Compare November 16, 2023 10:49
@catenacyber
Copy link
Collaborator Author

Rebased and force-pushed to get Suricata CI greener

@@ -54,12 +54,8 @@ checks:
- filter:
count: 1
match:
dest_ip: 192.168.200.1
Copy link
Member

Choose a reason for hiding this comment

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

it's unclear why this is removed... did something change in ssh logging?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not yet in OISF/suricata#9797 v13.1 but in v13.4 there is some change

Pushing a new SV PR without this, and rebasing Suricata PR to reference the new SV PR

@catenacyber
Copy link
Collaborator Author

Replaced by #1482

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

Successfully merging this pull request may close these issues.

2 participants