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

Deprecate Sigma status CSV usage from code #2913

Merged
merged 3 commits into from
Sep 29, 2023

Conversation

jaegeral
Copy link
Collaborator

@jaegeral jaegeral commented Sep 27, 2023

Removing some left overs from the old File based Sigma usage.

As the sigma rule status csv is not maintained, it is removed

If there is a new need / desire to track status of upstream Sigma rules and which work in Timesketch / which not, a new way needs to be established, a CSV does not play nice with the community.

@jaegeral jaegeral added Sigma Issued related to our Sigma integration Code Health Code health improvements labels Sep 27, 2023
@jaegeral jaegeral added this to the Release: 20231025 milestone Sep 27, 2023
@jaegeral jaegeral requested a review from jkppr September 27, 2023 09:35
@jaegeral jaegeral marked this pull request as ready for review September 27, 2023 09:35
@berggren berggren self-requested a review September 29, 2023 06:31
@berggren
Copy link
Contributor

If the CSV file has been deprecated can you add a note in it that says so? Then we can mark it for deletion in a couple of months.

@berggren berggren changed the title clean up a few Sigma left overs Deprecate Sigma status CSV Sep 29, 2023
@jaegeral jaegeral changed the title Deprecate Sigma status CSV Deprecate Sigma status CSV usage from code and move the status file to contrib Sep 29, 2023
@jaegeral
Copy link
Collaborator Author

If the CSV file has been deprecated can you add a note in it that says so? Then we can mark it for deletion in a couple of months.

clarified in the PR description, the usage of the file has been deprecated, the content of the file itself might still be valuable for me / others, thus moving it to contrib outside of the data folder, as the file itself is no longer used in the code.

@jaegeral jaegeral changed the title Deprecate Sigma status CSV usage from code and move the status file to contrib Deprecate Sigma status CSV usage from code Sep 29, 2023
@jaegeral jaegeral requested a review from jkppr September 29, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Health Code health improvements Sigma Issued related to our Sigma integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants