-
Notifications
You must be signed in to change notification settings - Fork 644
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 password autocomplete #4461
Conversation
@blowdart , looks good? |
What does the experience in the browser looks like when this attribute is not applied (i.e. current state in PROD)? |
Chrome for example lets you choose which password to apply in the field |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need a bug opened on feature-redesign
for this.
Created a bug Joel: https://github.com/NuGet/Engineering/issues/623 |
@skofman1 thanks!! |
@@ -319,12 +319,12 @@ | |||
@Html.ValidationSummaryFor("ChangePassword") | |||
<div class="form-field"> | |||
@Html.LabelFor(m => m.ChangePassword.OldPassword) | |||
@Html.PasswordFor(m => m.ChangePassword.OldPassword) | |||
@Html.PasswordFor(m => m.ChangePassword.OldPassword, new { autocomplete = "off" }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You sure this works? AFAIK, browsers are disregarding autocomplete=off
on password fields (IE11, FF, Chrome).
https://msdn.microsoft.com/library/ms533486
https://bugzilla.mozilla.org/show_bug.cgi?id=956906#c100
https://bugs.chromium.org/p/chromium/issues/detail?id=352347 https://bugs.chromium.org/p/chromium/issues/detail?id=587466
Fixes this: https://github.com/NuGet/Engineering/issues/622