Project

General

Profile

Bug #25527

Allow $ character in IPMI password

Added by Jeffrey Lockard almost 2 years ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Nice to have
Assignee:
William Grzybowski
Category:
GUI (new)
Target version:
11.1-RC3
Seen in:
11.0-U2
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Supermicro X10SL7-F
e3-1231 v3
32 Gb

ChangeLog Required:
No

Description

I was trying to get a handle on the IPMI interface in the FreeNAS GUI and lost outside access to IPMI via the direct IPMI/BMC ethernet port and address.

I was able to "fix" my outside access by using a password that did not use a "$" character in the FreeNAS GUI.

So, in summary, I have been a longtime user of IPMI independently of FreeNAS, no issues.
1. Set "Pa5$w0rd" in FreeNAS GUI --> lose outside access.
2. Set "Pa55w0rd" in FreeNAS GUI --> get outside access back.
3. Repeat with "hotdog" and "hotdog$" to verify trouble is repeatable and with the "$" character in the password.
4. Report bug.

Associated revisions

Revision d8aa4181 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/ipmi): no need to quote, no longer using shell

Ticket: #25527

Revision 461b2636 (diff)
Added by William Grzybowski over 1 year ago

fix(middlewared/ipmi): no need to quote, no longer using shell

Ticket: #25527
(cherry picked from commit d8aa41819d3bfc03af368e2f92181cc091b30eba)

History

#1 Updated by Bonnie Follweiler almost 2 years ago

  • Assignee changed from Release Council to William Grzybowski

#2 Updated by William Grzybowski almost 2 years ago

  • Status changed from Unscreened to Screened
  • Target version set to 11.1

Thanks for the report.

#3 Updated by Jeffrey Lockard almost 2 years ago

No problem, thanks for looking into this.

#4 Updated by William Grzybowski almost 2 years ago

  • Status changed from Screened to Ready For Release
  • Priority changed from No priority to Nice to have

Actually, I believe this has been fixed as part of the logic move to middlewared, using run() instead of shell Popen.
Fix will be available in 11.1

2961ae2e94

#5 Updated by Dru Lavigne almost 2 years ago

  • Subject changed from IPMI GUI interface page not setting passwords that have a "$" character to Allow $ character in IPMI password

#6 Updated by Dru Lavigne over 1 year ago

  • Target version changed from 11.1 to 11.1-BETA1

#7 Updated by Dru Lavigne over 1 year ago

  • Status changed from Ready For Release to Resolved

#8 Updated by Bonnie Follweiler over 1 year ago

  • QA Status Test Fails FreeNAS added
  • QA Status deleted (Not Tested)

I updated to FreeNAS-11.1-RC1
I logged on as Username:admin Password:admin
I changed the password to a$dmin and it gives me an "Invalid Authentication" error message
If you want to verify I have the machine availble for you to test

#9 Updated by Bonnie Follweiler over 1 year ago

  • Status changed from Resolved to Unscreened

#10 Updated by William Grzybowski over 1 year ago

  • Status changed from Unscreened to Screened
  • Target version changed from 11.1-BETA1 to 11.1

#11 Updated by William Grzybowski over 1 year ago

  • Status changed from Screened to Needs Developer Review
  • Assignee changed from William Grzybowski to Bartosz Prokop

#12 Updated by Bartosz Prokop over 1 year ago

  • Status changed from Needs Developer Review to Reviewed by Developer

#13 Updated by William Grzybowski over 1 year ago

  • Status changed from Reviewed by Developer to Ready For Release
  • Assignee changed from Bartosz Prokop to William Grzybowski

#14 Updated by William Grzybowski over 1 year ago

  • QA Status Not Tested added
  • QA Status deleted (Test Fails FreeNAS)

#15 Updated by Bonnie Follweiler over 1 year ago

  • Needs QA changed from Yes to No
  • QA Status Test Passes FreeNAS added
  • QA Status deleted (Not Tested)

#16 Updated by Dru Lavigne over 1 year ago

  • Target version changed from 11.1 to 11.1-RC2

#17 Updated by Dru Lavigne over 1 year ago

  • Target version changed from 11.1-RC2 to 11.1-RC3

#18 Updated by Dru Lavigne over 1 year ago

  • Status changed from Ready For Release to Resolved

Also available in: Atom PDF