-
Notifications
You must be signed in to change notification settings - Fork 36
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
Migrate from go-yara to yara-x; improve performance and readability #734
Merged
Merged
Changes from 1 commit
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
dcac602
Swap over to yara-x; improve performance and readability
egibs 7810b35
Merge branch 'main' into use-yara-x-take-2
egibs e792cc2
Refresh test data
egibs cbeb769
Small concurrency tweaks
egibs 0611ab5
Tweak third-party.yaml
egibs 0b3cee5
Re-use c.Concurrency
egibs c731b9d
Merge branch 'main' into use-yara-x-take-2
egibs 015dcf5
Merge branch 'main' into use-yara-x-take-2
egibs 5998611
Refresh test data
egibs fc6d6d1
Fix stats JSON test
egibs 05da5c8
Add UPX to Workflows
egibs f1fedf3
Replace empty match strings with the matched patterns
egibs 62c07b6
Revert str variable change
egibs 297bd1e
Use code highlighting to avoid escaping patterns
egibs 3832c87
Merge branch 'main' into use-yara-x-take-2
egibs a82d52e
Small comment tweaks
egibs d3d639a
Move scanner pool assignment in refresh functions
egibs 63746ef
One last go get -u ./...
egibs 9c11726
Split up test jobs
egibs 8a067ca
Merge branch 'main' into use-yara-x-take-2
egibs 280b87b
Merge branch 'main' into use-yara-x-take-2
egibs f7949ed
Another round of go get -u
egibs 980e2b2
Container hardening
egibs 6b20338
Remove --security-opt
egibs f627dd7
Only run container workloads in the parent repository
egibs 5b4980c
Update golangci-lint job
egibs 83dfa49
Use 4-core CPU shares
egibs 0270283
Clean up API installation instructions
egibs 734e965
Less redundant wording
egibs 8c78280
Simplify further
egibs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to wait until CI was running consistently before locking everything down.