Password Handling in SMS-Handler has two different maximum password length limits

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: wmathanaraj, Assigned: marta)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Impact: Passwords may not be as strong as they could be

Fix: Decide on one limit and use it throughout the code (privacy-panel.js:256, and remote_privacy_protection.js:49)

Additional note: Dynamically creating the regular expression with the RegExp constructor could allow removing double-checking for disabled commands
(Reporter)

Updated

4 years ago
Blocks: 1083953
(Assignee)

Comment 1

4 years ago
Created attachment 8507717 [details] [diff] [review]
bug_1083822.patch
Assignee: bugzilla → marta
Status: NEW → ASSIGNED
Attachment #8507717 - Flags: review?(fbraun)
Comment on attachment 8507717 [details] [diff] [review]
bug_1083822.patch

Review of attachment 8507717 [details] [diff] [review]:
-----------------------------------------------------------------

Yep
Attachment #8507717 - Flags: review?(fbraun) → review+
(Assignee)

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Blocks: 1088565
(Assignee)

Updated

4 years ago
No longer blocks: 1088565
You need to log in before you can comment on or make changes to this bug.