Bug 8724 - Add a 2nd password text field at the web security settings page
: Add a 2nd password text field at the web security settings page
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.1
: All All
: P5 enhancement (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-14 06:02 UTC by Markus Schiegl
Modified: 2009-09-08 09:23 UTC (History)
0 users

See Also:
Category: ---


Attachments
add 2nd pw text field + verify both pw fields (1.73 KB, patch)
2008-07-14 06:02 UTC, Markus Schiegl
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Schiegl 2008-07-14 06:02:24 UTC
Created attachment 3579 [details]
add 2nd pw text field + verify both pw fields

Because of http://forums.slimdevices.com/showthread.php?t=49827
1) I think it is a good idea to add a second verification field for the password
2) I thought it would match my limited perl skills to do it myself

so here it is: a patch (against 7.1) which adds a second text field and compares those two values before the password is stored into the pref file.

What do you think? crap? hmm? ok?

kind regards,
Markus
Comment 1 Blackketter Dean 2008-07-22 05:12:40 UTC
Michael:  What do you think?
Comment 2 Michael Herger 2008-07-22 05:34:41 UTC
Sounds reasonable - though not urgent. Shall we include it with 7.2 or not?
Comment 3 Blackketter Dean 2008-07-22 06:17:04 UTC
I've got no problem with it in 7.2, assuming everything else is done and you are bored. :)

Otherwise, 7.3 would be better.
Comment 4 Michael Herger 2008-07-22 07:37:12 UTC
don't like bugs targeted for future - they tend to go lost. Will punt post-7.2 if I can't reach boredom in time.
Comment 5 Michael Herger 2008-07-24 10:06:12 UTC
change 22070 - thanks!
Comment 6 Markus Schiegl 2008-07-26 03:25:22 UTC
Can you show me where i can find this change in svn, please?

It neither shows up in the webview nor have i a received a svn-commit-mail (svn update doesn't help either)

Was it commited to a non-public branch?

thanks,
Markus
Comment 7 Spies Steven 2008-08-27 11:49:34 UTC
Verified with SC 7.2 - 22900
Comment 8 James Richardson 2008-12-15 12:34:15 UTC
This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 9 Chris Owens 2009-07-31 10:24:46 UTC
Reduce number of active targets for SC