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.

Sign up for GitHub

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

Jump to bottom

Filestore 6388 v3 #1427

Closed
wants to merge 2 commits into from
Closed

Filestore 6388 v3 #1427

wants to merge 2 commits into from

Conversation

catenacyber
Copy link
Collaborator

@catenacyber
Adds a test about filestore
7b1d89d 
That it does not store too many files
@catenacyber
Adds a test for filestore keyword with one direction
3ebe667 
And a flow scope, to check that the files are indeed stored
@catenacyber catenacyber mentioned this pull request Oct 16, 2023
Closed
@catenacyber catenacyber added the requires suricata pr Depends on a PR in Suricata label Oct 26, 2023
@victorjulien
Copy link
Member

I see these fail for master too, which means that the fix should first be towards master and then we'll consider the backport side.

@catenacyber catenacyber mentioned this pull request Dec 7, 2023
Closed
@catenacyber catenacyber marked this pull request as draft December 7, 2023 09:38
@catenacyber
Copy link
Collaborator Author

#1522 should go in first before considering the other test

@catenacyber
Copy link
Collaborator Author

Closing as master6 goes EOL soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
requires suricata pr Depends on a PR in Suricata
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@catenacyber @victorjulien