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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants