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: reapply: remove default usage of parse-arguments #4544

Merged
merged 3 commits into from
Jan 23, 2025

Conversation

geyslan
Copy link
Member

@geyslan geyslan commented Jan 22, 2025

Close: #4541

This reverts #4331 back.

1. Explain what the PR does

c922aaa Reapply "feat(sigs): refactor to use nonparsed arguments"
f680998 Revert "Add workaround: Revert to using raw ... argument values in engine stage".
1f00c68 chore(deps): bump signatures/helpers to f57a1eb

c922aaa Reapply "feat(sigs): refactor to use nonparsed arguments"

This reverts commit b46a2b6bde588825877ea45ec690a7dc871fe73c.

f680998 Revert "Add workaround: Revert to using raw ... argument values in engine stage".

This reverts commit a481d11d2ad3eda1ea9c5a187a8574e0c46e9e33.

2. Explain how to test it

3. Other comments

@geyslan geyslan self-assigned this Jan 22, 2025
@geyslan geyslan changed the title feat: remove default usage of parse-arguments feat: reapply: remove default usage of parse-arguments Jan 22, 2025
@geyslan geyslan requested a review from yanivagman January 22, 2025 19:19
@geyslan geyslan marked this pull request as ready for review January 22, 2025 19:19
@geyslan geyslan requested a review from rscampos January 22, 2025 19:57
@geyslan geyslan modified the milestone: v0.23.0 Jan 22, 2025
Copy link
Collaborator

@yanivagman yanivagman left a comment

Choose a reason for hiding this comment

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

LGTM once rebased on top of #4545

@geyslan geyslan force-pushed the 4541-revert-4461 branch 2 times, most recently from 82ac6b6 to 17aa0ea Compare January 23, 2025 10:33
@geyslan
Copy link
Member Author

geyslan commented Jan 23, 2025

/fast-forward

@github-actions github-actions bot merged commit c922aaa into aquasecurity:main Jan 23, 2025
36 checks passed
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.

Revert back to use raw arguments for threat detection events
2 participants