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

Fix management of custom rule and decoders files #5734

Merged
merged 6 commits into from
Aug 11, 2023

Conversation

Desvelao
Copy link
Member

@Desvelao Desvelao commented Jul 24, 2023

Description

This pull request fixes the management of custom rule or decoder file through the relative directory name.

TODO:

Issues Resolved

#5176

Evidence

[Provide screenshots or videos to prove this PR solves the issues]

Test

Wazuh dashboard

Legend:
⚫: none
🟢: pass
🟡: warning
🔴: fail
⚪: not applicable

UI

Test Chrome Firefox Safari
Rules: Create a new file. The file should be saved on etc/rules directory.
Rules: Import a new file with overwrite enabled. The file should be saved on etc/rules directory and should overwrite the previous file if it exists.
Rules: Import a new file with overwrite disabled. The file should be saved on etc/rules directory and should not overwrite the previous file if it exists.
Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer.
Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer.
Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.
Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated.
Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated.
Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted.
Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted.
Decoders: Create a new file. The file should be saved on etc/decoders directory.
Decoders: Import a new file with overwrite enabled. The file should be saved on etc/decoders directory and should overwrite the previous file if it exists.
Decoders: Import a new file with overwrite disabled. The file should be saved on etc/decoders directory and should not overwrite the previous file if it exists.
Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer.
Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer.
Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.
Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated.
Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated.
Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted.
Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted.

Details

⚫ Rules: Create a new file. The file should be saved on `etc/rules` directory.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Import a new file with overwrite enabled. The file should be saved on `etc/rules` directory and should overwrite the previous file if it exists.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Import a new file with overwrite disabled. The file should be saved on `etc/rules` directory and should not overwrite the previous file if it exists.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Create a new file. The file should be saved on `etc/decoders` directory.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Import a new file with overwrite enabled. The file should be saved on `etc/decoders` directory and should overwrite the previous file if it exists.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Import a new file with overwrite disabled. The file should be saved on `etc/decoders` directory and should not overwrite the previous file if it exists.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

⚫ Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted.

Chrome - ⚫

Firefox - ⚫

Safari - ⚫

Check List

  • All tests pass
    • yarn test:jest
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@Desvelao Desvelao requested a review from a team as a code owner July 24, 2023 15:23
@Desvelao Desvelao self-assigned this Jul 24, 2023
@Desvelao Desvelao marked this pull request as draft July 24, 2023 15:24
@asteriscos asteriscos marked this pull request as ready for review August 10, 2023 14:45
@Tostti
Copy link
Member

Tostti commented Aug 10, 2023

Legend:
⚫: none
🟢: pass
🟡: warning
🔴: fail
⚪: not applicable

UI

Test Chrome Firefox Safari
Rules: Create a new file. The file should be saved on etc/rules directory. 🟢
Rules: Import a new file with overwrite enabled. The file should be saved on etc/rules directory and should overwrite the previous file if it exists. 🟢
Rules: Import a new file with overwrite disabled. The file should be saved on etc/rules directory and should not overwrite the previous file if it exists. 🟢
Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer. 🟢
Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer. 🟢
Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager. 🟢
Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated. 🟢
Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated. 🟢
Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted. 🟢
Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted. 🟢
Decoders: Create a new file. The file should be saved on etc/decoders directory. 🟢
Decoders: Import a new file with overwrite enabled. The file should be saved on etc/decoders directory and should overwrite the previous file if it exists. 🟢
Decoders: Import a new file with overwrite disabled. The file should be saved on etc/decoders directory and should not overwrite the previous file if it exists. 🟢
Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer. 🟢
Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer. 🟢
Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager. 🟢
Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated. 🟢
Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated. 🟢
Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted. 🟢
Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted. 🟢

Details

🟢 Rules: Create a new file. The file should be saved on `etc/rules` directory.

Chrome - 🟢
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Import a new file with overwrite enabled. The file should be saved on `etc/rules` directory and should overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Import a new file with overwrite disabled. The file should be saved on `etc/rules` directory and should not overwrite the previous file if it exists.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Create a new file. The file should be saved on `etc/decoders` directory.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Import a new file with overwrite enabled. The file should be saved on `etc/decoders` directory and should overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Import a new file with overwrite disabled. The file should be saved on `etc/decoders` directory and should not overwrite the previous file if it exists.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

@lucianogorza
Copy link
Contributor

lucianogorza commented Aug 10, 2023

Legend:
⚫: none
🟢: pass
🟡: warning
🔴: fail
⚪: not applicable

UI

Test Chrome Firefox Safari
Rules: Create a new file. The file should be saved on etc/rules directory. 🟢 🟢
Rules: Import a new file with overwrite enabled. The file should be saved on etc/rules directory and should overwrite the previous file if it exists. 🟢 🟢
Rules: Import a new file with overwrite disabled. The file should be saved on etc/rules directory and should not overwrite the previous file if it exists. 🟢 🟢
Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer. 🟢 🟢
Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer. 🟢 🟢
Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager. 🟢 🟢
Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated. 🟢 🟢
Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated. 🟢 🟢
Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted. 🟢
Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted. 🟢
Decoders: Create a new file. The file should be saved on etc/decoders directory. 🟢
Decoders: Import a new file with overwrite enabled. The file should be saved on etc/decoders directory and should overwrite the previous file if it exists. 🟢
Decoders: Import a new file with overwrite disabled. The file should be saved on etc/decoders directory and should not overwrite the previous file if it exists. 🟢
Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer. 🟢
Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer. 🟢
Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager. 🟢
Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated. 🟢
Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated. 🟢
Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted. 🟢
Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted. 🟢

Details

🟢 Rules: Create a new file. The file should be saved on `etc/rules` directory.

Chrome - 🟢
image
image

Firefox - 🟢
image
image

Safari - ⚫

🟢 Rules: Import a new file with overwrite enabled. The file should be saved on `etc/rules` directory and should overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - 🟢
image
image

Safari - ⚫

🟢 Rules: Import a new file with overwrite disabled. The file should be saved on `etc/rules` directory and should not overwrite the previous file if it exists.

Chrome - 🟢
image
image
image

Firefox - 🟢
image
image
image

Safari - ⚫

🟢 Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - :🟢
image
image

Safari - ⚫

🟢 Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - 🟢
image
image

Safari - ⚫

🟢 Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

image

Chrome - 🟢
image
image

Firefox - 🟢
image
image

Safari - ⚫

🟢 Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox - 🟢
image
image

Safari - ⚫

🟢 Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox -🟢
image
image

Safari - ⚫

🟢 Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted.

Chrome - ⚫

Firefox - 🟢
image
image
image

Safari - ⚫

🟢 Decoders: Create a new file. The file should be saved on `etc/decoders` directory.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Import a new file with overwrite enabled. The file should be saved on `etc/decoders` directory and should overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Import a new file with overwrite disabled. The file should be saved on `etc/decoders` directory and should not overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

image
image

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

@lucianogorza lucianogorza self-requested a review August 10, 2023 18:28
Tostti
Tostti previously approved these changes Aug 11, 2023
Copy link
Member

@Tostti Tostti left a comment

Choose a reason for hiding this comment

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

TEST:✔️
CR:✔️

LGTM

@JuanGarriuz
Copy link
Member

JuanGarriuz commented Aug 11, 2023

Legend:
⚫: none
🟢: pass
🟡: warning
🔴: fail
⚪: not applicable

UI

Test Chrome Firefox Safari
Rules: Create a new file. The file should be saved on etc/rules directory. 🟢
Rules: Import a new file with overwrite enabled. The file should be saved on etc/rules directory and should overwrite the previous file if it exists. 🟢
Rules: Import a new file with overwrite disabled. The file should be saved on etc/rules directory and should not overwrite the previous file if it exists. 🟢
Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer. 🟢
Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer. 🟢
Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager. 🟢
Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated. 🟢
Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated. 🟢
Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted. 🟢
Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted. 🟢
Decoders: Create a new file. The file should be saved on etc/decoders directory. 🟢
Decoders: Import a new file with overwrite enabled. The file should be saved on etc/decoders directory and should overwrite the previous file if it exists. 🟢
Decoders: Import a new file with overwrite disabled. The file should be saved on etc/decoders directory and should not overwrite the previous file if it exists. 🟢
Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer. 🟢
Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer. 🟢
Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager. 🟢
Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated. 🟢
Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated. 🟢
Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted. 🟢
Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted. 🟢

Details

🟢 Rules: Create a new file. The file should be saved on `etc/rules` directory.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Import a new file with overwrite enabled. The file should be saved on `etc/rules` directory and should overwrite the previous file if it exists.

Chrome - 🟢

image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Import a new file with overwrite disabled. The file should be saved on `etc/rules` directory and should not overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Read the file content of a file located in ruleset/rules. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Read the file content of a file located in etc/rules/local_rules.xml. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Read the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Update the file content of a file located in etc/rules/local_rules.xml. Its content should be updated.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Update the file content of a file located in etc/rules/custom_directory/local_rules.xml. Its content should be updated.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Delete the file content of a file located in etc/rules/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Rules: Delete the file content of a file located in etc/rules/custom_directory/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Create a new file. The file should be saved on `etc/decoders` directory.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Import a new file with overwrite enabled. The file should be saved on `etc/decoders` directory and should overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Import a new file with overwrite disabled. The file should be saved on `etc/decoders` directory and should not overwrite the previous file if it exists.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in ruleset/decoders. Its content should be displayed in the rule viewer.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in etc/decoders/local_decoders.xml. Its content should be displayed in the rule viewer.

Chrome - 🟢

image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Read the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be displayed in the rule viewer. You should have a rule file in that directory, this could require configuration in the manager.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Update the file content of a file located in etc/decoders/local_decoders.xml. Its content should be updated.

Chrome - 🟢
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Update the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. Its content should be updated.

Chrome - 🟢
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Delete the file content of a file located in etc/decoders/local_rules.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

🟢 Decoders: Delete the file content of a file located in etc/decoders/custom_directory/local_decoders.xml. The file should be deleted.

Chrome - 🟢
image
image
image

Firefox - ⚫

Safari - ⚫

JuanGarriuz
JuanGarriuz previously approved these changes Aug 11, 2023
Copy link
Member

@JuanGarriuz JuanGarriuz left a comment

Choose a reason for hiding this comment

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

LGTM
Test: #5734 (comment)

lucianogorza
lucianogorza previously approved these changes Aug 11, 2023
Copy link
Contributor

@lucianogorza lucianogorza left a comment

Choose a reason for hiding this comment

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

TEST:✔️
CR:✔️

LGTM

@Desvelao Desvelao dismissed stale reviews from lucianogorza and JuanGarriuz via 5d16e9b August 11, 2023 14:13
@Tostti Tostti merged commit 8500a13 into 4.6.0 Aug 11, 2023
@Tostti Tostti deleted the fix/5176-manage-rules-decoders-file branch August 11, 2023 14:13
@github-actions
Copy link
Contributor

Code coverage (Jest) % values
Statements 11.48% ( 3959 / 34486 )
Branches 7.57% ( 1775 / 23422 )
Functions 10.86% ( 896 / 8246 )
Lines 11.66% ( 3847 / 32977 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use new default_ruleset parameter when reading rules and decoders file content
5 participants