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

[Backport 4.4-7.16] Fix password input in deploy new agent section #5110

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

github-actions[bot]
Copy link
Contributor

Backport b3b0222 from #5098.

* register agent section: password

* changelog

* fixed solaris, hp and alpine password input and added some styles

* fixed the textToCopy error displayed in the console

(cherry picked from commit b3b0222)
@github-actions github-actions bot requested a review from a team as a code owner January 12, 2023 09:00
Copy link
Member

@Desvelao Desvelao left a comment

Choose a reason for hiding this comment

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

review:

Kibana 7.16

Legend:
⚫: none
🟢: passed
🔴: failed
🟡: warning
⚪: not applicable

Test Chrome Package
Check that the 'Wazuh password' input does not appear in the deploy new agent section. 🟢 Kibana 7.16
Control that depending on whether the switch is activated or not the hidden password is shown with **** or the previously defined password is shown. 🟢 Kibana 7.16
Try to choose HP, Solaris or Alpine, check that the variable WAZUH_REGISTRATION_PASSWORD='hello' does not appear in the install command and that the 'show password' switch does not appear either. 🟢 Kibana 7.16
Check that the 'Wazuh password' entry appears in the deploy new agent section and that you can enter a password without having selected an os yet. 🟢 Kibana 7.16
Check that depending on whether the switch is activated or not the hidden password is shown with **** or the password you typed in is shown 🟢 Kibana 7.16
Choose some operating system other than hp, alpine or solaris put a password, check that the hide password switch works fine 🟢 Kibana 7.16
Choose some operating system other than HP, Alpine or Solaris set a password, check that the password hiding switch works fine, now choose HP, Solaris or Alpine check that the ui is not broken. 🟢 Kibana 7.16
Try choosing hp, solaris or alpine, the password input and the switch should not appear. 🟢 Kibana 7.16

Details

🟢 Check that the 'Wazuh password' input does not appear in the deploy new agent section.

Chrome - 🟢

image

🟢 Control that depending on whether the switch is activated or not the hidden password is shown with **** or the previously defined password is shown.

Chrome - 🟢

image
image

🟢 Try to choose HP, Solaris or Alpine, check that the variable WAZUH_REGISTRATION_PASSWORD='hello' does not appear in the install command and that the 'show password' switch does not appear either.

Chrome - 🟢

image
image
image

🟢 Check that the 'Wazuh password' entry appears in the deploy new agent section and that you can enter a password without having selected an os yet.

Chrome - 🟢

image

⚫ Check that depending on whether the switch is activated or not the hidden password is shown with **** or the password you typed in is shown

Chrome - ⚫

image
image

🟢 Choose some operating system other than hp, alpine or solaris put a password, check that the hide password switch works fine

Chrome - 🟢

image
image

🟢 Choose some operating system other than HP, Alpine or Solaris set a password, check that the password hiding switch works fine, now choose HP, Solaris or Alpine check that the ui is not broken.

Chrome - 🟢

image
image

🟢 Try choosing hp, solaris or alpine, the password input and the switch should not appear.

Chrome - 🟢

image
image
image

Copy link
Member

@asteriscos asteriscos left a comment

Choose a reason for hiding this comment

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

Test: ✔️

Without password setup

Peek.2023-01-12.13-19.mp4

With password setup

Peek.2023-01-12.13-24.mp4

With predefined password in /var/ossec/etc/authd.pass

Peek.2023-01-12.13-30.mp4

@Desvelao Desvelao merged commit 8a72a5c into 4.4-7.16 Jan 12, 2023
@Desvelao Desvelao deleted the backport-5098-to-4.4-7.16 branch January 12, 2023 12:32
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.

The "Deploy a new agent" page breaks if password authentication is enabled
3 participants