Logout alert not always correct.

RESOLVED FIXED in mozilla0.9.6

Status

SeaMonkey
Passwords & Permissions
RESOLVED FIXED
17 years ago
14 years ago

People

(Reporter: John Unruh, Assigned: Stephen P. Morse)

Tracking

Trunk
mozilla0.9.6
x86
All

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

1.05 KB, patch
Brian Ryner (not reading)
: review+
Alec Flett
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

17 years ago
1.) With a new profile, enter a name and password into http://mail.yahoo.com.
2.) Elect to save the info.
3.) Tasks>Privacy>Password Manager>Log out.
What happens: An alert states "Your sensitive information is now locked"
What is expected: Since I was never logged in, and the sensitive data is only 
obscured, I should not get this alert, or better, the words "Log out" would not 
be enabled.

Comment 1

17 years ago
I think this is password manager code.
Assignee: ssaux → morse
Component: Client Library → Password Manager
Product: PSM → Browser
QA Contact: junruh → tpreston
Version: 2.1 → other
(Assignee)

Comment 2

17 years ago
There are two different state variables here, and the wording doesn't make that 
clear.

1. Encrypted versus Obscurred

   If you have any saved sensitive information, is it encrypt or is it obscurred

2. Logged In versus Logged Out

   If your sensitive information is encrypted or you have any certs, can you get
   to them

The "logged out" and "locked" refer to the second state variable and has nothing 
to do with whether your sensitive information happens to be currently encrypted 
or obscured.

I mention all this as an argument against the reporter's statement that "logout" 
should not be enabled when the data is not encrypted.

So the correct wording on this dialog should be:

   "You are now logged out and will not be able to access any
    encrypted information"
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.6
(Assignee)

Comment 3

17 years ago
Created attachment 54764 [details] [diff] [review]
fixes the wording
(Assignee)

Comment 4

17 years ago
cc'ing law and alecf for reviews

Comment 5

17 years ago
Comment on attachment 54764 [details] [diff] [review]
fixes the wording

sr=alecf
Attachment #54764 - Flags: superreview+
Comment on attachment 54764 [details] [diff] [review]
fixes the wording

r=bryner
Attachment #54764 - Flags: review+
(Assignee)

Comment 7

17 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.