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

Update the help command in password management with the required options when trying to change the API user password #7412

Conversation

Enaraque
Copy link
Member

Related Issue
https://github.com/wazuh/external-devel-requests/issues/2301

Description

In the wazuh-password-tool.sh script, to change the password of an api user, it is necessary for the api user to specify the --api option. Failure to do so would result in the script's help being displayed.
To this end, a message has been added to the help in the -au and -ap options saying that it is necessary to specify the -A|--api option when using those options.

Tests

Captura de pantalla 2024-06-17 a las 13 21 23

Checks

Docs building

  • Compiles without warnings.

Code formatting and web optimization

  • Uses three spaces indentation.
  • Adds or updates meta descriptions accordingly.
  • Updates the redirects.js script if necessary (check this guide).

Writing style

  • Uses present tense, active voice, and semi-formal registry.
  • Uses short, simple sentences.
  • Uses bold for user interface elements, italics for key terms or emphasis, and code font for Bash commands, file names, REST paths, and code.

@Enaraque Enaraque self-assigned this Jun 18, 2024
Copy link
Member

@rauldpm rauldpm left a comment

Choose a reason for hiding this comment

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

LGTM

@rauldpm rauldpm changed the title Update help command in password management with the requiere options when trying to change the API user password Update the help command in password management with the required options when trying to change the API user password Jun 18, 2024
@rauldpm rauldpm merged commit 5bb031b into 4.8.1 Jun 18, 2024
3 checks passed
@rauldpm rauldpm deleted the change/2301-password-tool-silently-fails-on-single-user-password-change branch June 18, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants