Project

General

Profile

Bug #24740

Avatar?id=14398&size=22x22

Properly quote password created during installation

Added by Grey Golla almost 2 years ago. Updated 11 months ago.

Status:
Done
Priority:
Nice to have
Assignee:
Kris Moore
Category:
OS
Target version:
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

When first installing freenas 11 (and 9 I believe), setting the root password truncates it to the first space. For example, entering "the quick brown fox" causes the root password to be set to "the" instead.
When changing the root password later through the console utility, passwords containing spaces are handled correctly.

Associated revisions

Revision 848a9da3 (diff)
Added by Kris Moore over 1 year ago

Go through and do some proper quoting for passwords so we can
get passwords with whitespace properly set at install time.

Ticket: #24740

Revision d4dcce19 (diff)
Added by Kris Moore over 1 year ago

Go through and do some proper quoting for passwords so we can (#449)

get passwords with whitespace properly set at install time.

Ticket: #24740

History

#1 Updated by Sean Fagan almost 2 years ago

  • Status changed from Unscreened to Screened

The current mechanism does not allow for spaces (or some other symbols). I suppose it could warn about that.

#2 Updated by Dru Lavigne over 1 year ago

  • Status changed from Screened to 15
  • Assignee changed from Sean Fagan to Kris Moore
  • Target version set to 11.2-BETA1

Should this be punted for the new GUI to dtrt?

#3 Avatar?id=14398&size=24x24 Updated by Kris Moore over 1 year ago

  • Status changed from 15 to Fix In Progress
  • Priority changed from No priority to Nice to have

#4 Avatar?id=14398&size=24x24 Updated by Kris Moore over 1 year ago

  • Status changed from Fix In Progress to Needs Developer Review
  • Assignee changed from Kris Moore to Brandon Schneider

https://github.com/freenas/freenas/pull/449

Brandon, you are crazy enough like me to read / like shell. Care to do a quick review?

#5 Updated by Brandon Schneider over 1 year ago

  • Status changed from Needs Developer Review to Reviewed by Developer
  • Assignee changed from Brandon Schneider to Kris Moore

Was reviewed many moons ago, cleaning up ticket.

#6 Updated by Dru Lavigne over 1 year ago

  • Status changed from Reviewed by Developer to Ready For Release

#7 Updated by Dru Lavigne about 1 year ago

  • Subject changed from Root password truncated at first space in initial setup to Properly quote password created during installation
  • Status changed from Ready For Release to Done
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#8 Updated by Dru Lavigne 12 months ago

  • Status changed from Done to Ready for Testing

#9 Updated by Ethan Large 11 months ago

  • Status changed from Ready for Testing to Passed Testing
  • Severity set to New
  • Needs QA changed from Yes to No

Tested with FreeNAS-11.2-MASTER-201806060445-62522c5.iso.

Root password can now successfully be set to a string containing spaces, without truncation.

#10 Updated by Dru Lavigne 11 months ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF