Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix the potential vulnerability of password showcase of external providers #969

Merged

Conversation

saksham-oracle
Copy link
Contributor

@saksham-oracle saksham-oracle commented Oct 17, 2024

Changes introduced with this PR

Issue: Password was visible for external providers after changing input type from password to text in browser developer tools (Inspect tools).

Fix: Added the logic for sending the ******* as password text to UI and updating the password only if user makes any change in password.

Fixes CVE-2024-7259

Are you the owner of the code you are sending in, or do you have permission of the owner?

y

…iders

CVE-2024-7259

Issue: Password was visible for external providers after changing input type from password to text in browser developer tools (Inspect tools).

Fix: Added the logic for sending the ******* as password text to UI and updating the password only if user makes any change in password.

Signed-off-by: Saksham Srivastava <saksham.sa.srivastava@oracle.com>
@sandrobonazzola sandrobonazzola added this to the ovirt-4.5.7 milestone Oct 18, 2024
@sandrobonazzola sandrobonazzola removed the request for review from sgratch October 18, 2024 07:49
@sandrobonazzola sandrobonazzola merged commit 4ad9306 into oVirt:master Oct 18, 2024
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants