"Login not secure" on extension pages collecting sensitive information
Reported by
andresan...@gmail.com,
Jun 20 2017
|
|||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3135.4 Safari/537.36 Steps to reproduce the problem: In this case, LastPass is being used. I believe this would affect any other extension pages using fields asking for sensitive information such as passwords. 1. Install the extension 2. Log into LastPass 3. Open the vault (will open up the extension page) 4. Add or edit either a site or a secure note 5. Press on the password field (if any) What is the expected behavior? What went wrong? The "login not secure" message shows up before the autofill suggestions. Attached is a screenshot of what this would look like. WebStore page: https://chrome.google.com/webstore/detail/lastpass-free-password-ma/hdokiejnpimakedhajhdlcegeplioahd Did this work before? N/A Chrome version: 61.0.3135.4 Channel: dev OS Version: 10.0 Flash Version: When it comes to password manager extensions, it may deter users from managing passwords and other sensitive information via extension page(s). Especially when it's safe to enter such details. This could be to help avoid users from being phished by malicious extensions added in via third party software (e.g. bundleware). Yes, there's worse but that's always a possibility. Any thoughts on this?
,
Jun 20 2017
,
Jun 22 2017
Somehow, I am not seeing the Login not secure message when editing gmail password on the latest canary(61.0.3138.0) and the reported version: 61.0.3135.4 on Windows-10. andresantosuk@: Could you please confirm if anything being missed here in repro steps?
,
Jun 22 2017
I took some time to narrow this down in a virtual machine. With a fresh install without syncing and installing the extension via Chrome Store, it works fine. Fresh w/ extension sync only: no warning Fresh w/ extension & password sync only: warning (also says "use password for:" followed by a list of emails and usernames) So from what I can tell so far, you would need to have Chrome remember your login credentials (and have password sync enabled). It should then end up on the list that would show up when interacting with password field(s). The virtual machine has been using the exact same browser that was used on the host (61.0.3135.4 Dev x64).
,
Jun 22 2017
Thank you for providing more feedback. Adding requester "ajha@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 29 2017
Unable to reproduce this issue on Windows 10 with chrome dev #61.0.3135.4, Canary #61.0.3143.0, followed the steps as mentioned in comment #0 andresantosuk@ could you retry the same scenario on clean profile with no apps/extensions(except "Last Pass") and let us know you your observations.
,
Jun 29 2017
In comment #5, there is more information of how to come along this issue (as soon as it was discovered what was in #0 wasn't enough -- my bad for not properly checking with a fresh install). It involves the passwords being synchronised. Just to clarify: are you asking me to go fresh, log in, synchronise everything apart from extensions, install the affected extension via Chrome Store and then see if it's still an issue?
,
Jun 29 2017
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 4 2017
andresantosuk@ As per comment #8 create a new profile and install only "LastPass" extension and synchronize , then re-try the scenario and let us know your observations.
,
Jul 4 2017
Version: 61.0.3141.7 (Official Build) dev (64-bit) (cohort: Dev) Tested in a Windows 7 virtual machine (same one used in comment #5). This requires the user to have their username and/or password fields on the extension page to show up as a light tint of yellow (RGB 250, 255, 189). It would indicate that the user can select an email/username that would use the password associated with that selection. 1. Install Chrome Dev as fresh 2. Visit and install the LastPass extension from the Chrome Web Store 3. Log into Chrome Dev but set to allow altering of syncronisation settings (there is a checkbox for it after successful login) before it begins synchronising 4. Disable the synchronising of everything apart from 'password' sync 5. Navigate elsewhere in the settings to get it to start synchronising 6. Log into LastPass 7. Visit LastPass vault extension page 8. Access site/note management popup (for editing sites or notes) 9. Press either on the username or password field Result: "Login not secure" warning + list of emails/usernames. But there is more. This happens depending on the order of how things are done and it could sometimes be random(?). Let me expand on that.. We will begin with these steps. It is basically the same as above but re-ordered. 1. Install Chrome Dev as fresh 2. Visit and install the LastPass extension from the Chrome Web Store 3. Log into LastPass 4. Log into Chrome Dev but set to allow altering of syncronisation settings (there is a checkbox for it after successful login) before it begins synchronising 5. Disable the synchronising of everything apart from 'password' sync 6. Navigate elsewhere in the settings to get it to start synchronising 7. Visit LastPass vault extension page 8. Access site/note management popup (for editing sites or notes) Result: List of emails/usernames but no "login not secure" warning. As for /random/.. On 22nd June 2017, while I was doing a bit of testing, I once ended up having the warning but no list displaying. The steps to reproduce had not been documented.
,
Jul 4 2017
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 5 2017
As TE team was unable to reproduce the issue, requesting Extensions dev team to look into this issue and update accordingly.
,
Oct 23 2017
I'm guessing that this doesn't reproduce anymore, and because the warning feature isn't launching, we can probably just close this bug now.
The PasswordAutofillAgent::ShouldShowNotSecureWarning function has:
return security_state::IsHttpWarningInFormEnabled() &&
!content::IsOriginSecure(
url::Origin(render_frame()->GetWebFrame()->Top()->GetSecurityOrigin()).GetURL());
}
IsOriginSecure should be returning true for chrome-extension, because in ChromeContentClient::AddAdditionalSchemes, we call
schemes->secure_schemes.push_back(extensions::kExtensionScheme);
,
Oct 23 2017
Even though this had been closed (with no intention to have it reopened), I'd like to confirm that this had been resolved. Currently using v63.0.3239.9 Dev channel Win64 but likely fixed some time ago. Thanks. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by andresan...@gmail.com
, Jun 20 20172.4 KB
2.4 KB View Download