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

Agent centralized: pass the argument list to the modules to execute commands #234

Open
Nicogp opened this issue Oct 22, 2024 · 1 comment
Assignees
Labels

Comments

@Nicogp
Copy link
Member

Nicogp commented Oct 22, 2024

Parent issue:

Description

As part of the development of #32 it is necessary that the module receives the list of command execution parameters, in it will come the list of groups.

To do this, the ExecuteCommand method of the ModuleWrapper must be changed so that in addition to passing the command, the list of parameters is also passed

In addition, it may be necessary to make changes to the way the command parameters obtained from the command queue (GetCommandFromQueue method) are parsed.

@Nicogp
Copy link
Member Author

Nicogp commented Oct 23, 2024

(2024/10/22)

  • I reviewed the changes that @jr0me started to make
  • I continued with the changes, I changed the parse mode of the command parameters.

(2024/10/23)

  • I created a new branch with the changes, accommodated the commits
  • I'm fixing the tests that fail due to the applied changes

(2024/10/25)

  • Resumed development
  • I rebased with the last changes
  • Added the parameters to the ExecuteCommand function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In progress
Development

When branches are created from issues, their pull requests are automatically linked.

1 participant