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

tests/luaxform: Lua transform tests #2090

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jlucovsky
Copy link
Contributor

@jlucovsky jlucovsky commented Oct 10, 2024

Continuation of #2044

This commit adds tests for new Lua transform luaxform

  • Basic transform operation
  • Ensure non-existent Lua scripts are detected
  • Ensure Lua scripts without transform functions are detected
  • Ensure Lua scripts properly receive optional transform arguments

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/2290
Suricata PR: OISF/suricata#11930

jufajardini and others added 2 commits October 10, 2024 09:46
This commit adds tests for new Lua transform
- Basic transform operation
- Ensure non-existent Lua scripts are detected
- Ensure Lua scripts without transform functions are detected
- Ensure Lua scripts properly receive optional transform arguments
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.

2 participants