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

Don't allow to set same command sections in the configuration #963

Closed
chemamartinez opened this issue Jul 17, 2018 · 1 comment
Closed
Labels
duplicated This issue or pull request already exists type/bug Something isn't working
Projects

Comments

@chemamartinez
Copy link
Contributor

No error is shown when it is set two same <command> tags for active-response in the configuration file. For example:

<command>
    <name>disable-account</name>
    <executable>disable-account.sh</executable>
    <expect>user</expect>
    <timeout_allowed>yes</timeout_allowed>
  </command>

  <command>
    <name>disable-account</name>
    <executable>disable-account.sh</executable>
    <expect>user</expect>
    <timeout_allowed>yes</timeout_allowed>
  </command>

However, only the first one works so it should not be allowed to set two commands with the same name or at least it should trigger a warning message.

@chemamartinez chemamartinez added the type/bug Something isn't working label Jul 17, 2018
@chemamartinez chemamartinez added this to To do in Wazuh TBD via automation Jul 17, 2018
@TJOSERAFAEL TJOSERAFAEL removed this from To do in Wazuh TBD Aug 8, 2018
@TJOSERAFAEL TJOSERAFAEL added this to To do in Wazuh 3.6 via automation Aug 8, 2018
@vikman90 vikman90 added this to To do in Wazuh 3.7 via automation Aug 13, 2018
@vikman90 vikman90 removed this from To do in Wazuh 3.6 Aug 13, 2018
@vikman90 vikman90 added the duplicated This issue or pull request already exists label Aug 22, 2018
@vikman90
Copy link
Member

This issue is a duplicate of #1057
Fixed in #1128

Wazuh 3.7 automation moved this from To do to Done Aug 22, 2018
@vikman90 vikman90 added this to To do in Wazuh 3.5.1 via automation Aug 22, 2018
@vikman90 vikman90 removed this from Done in Wazuh 3.7 Aug 22, 2018
@vikman90 vikman90 moved this from To do to Done in Wazuh 3.5.1 Aug 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicated This issue or pull request already exists type/bug Something isn't working
Projects
No open projects
Wazuh 3.5.1
  
Done
Development

No branches or pull requests

2 participants