Project

General

Profile

Bug #45387

Add tooltip warning users to avoid using uppercase usernames when creating an SMB home share

Added by Matt Bunce about 2 years ago. Updated almost 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Samantha Bonham
Category:
Documentation
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

Using FreeNAS 11.2 BETA2 it seems like the full path to a user home directory need to be lowercase. Adding any capital letters seems to brake things - even though the path selected on the "edit user" page does save the path with capital letters.

I experienced this error when I...

Created a user called Matt, with no home directory set
Created a unix dataset called "homes"
Created a Windows dataset called "Matt" underneath "homes"
Set owner user and group to "Matt" for this new dataset
Edited the Matt user and set the home directory to "/mnt/vol/homes/Matt"
Created an SMB share called homes and set it as a home share
Restarted SMB service
Tried to navigate to \\FREENAS\Matt in Windows (path not found error)
Look at /var/log/samba4/log.smbd and you can see samba was trying to access "/mnt/vol/homes/matt"

As you can see, for some reason Samba is trying to access the correct home path, but with all lowercase characters.

If this can't be changed, then an error message when selecting the user home directory would be helpful.


Related issues

Related to FreeNAS - Bug #51590: New user creation rules only lower case allowedClosed

History

#1 Updated by Dru Lavigne about 2 years ago

  • Private changed from No to Yes
  • Reason for Blocked set to Need additional information from Author

Matt: please reproduce then attach a debug (System -> Advanced -> Save debug) to this ticket.

#2 Updated by Matt Bunce about 2 years ago

  • File debug.7z added

#3 Updated by Matt Bunce about 2 years ago

I have added the debug. I had to extract it first and hide some details before re-packing as a .7z

I hope that is OK?

#4 Updated by Dru Lavigne about 2 years ago

  • Category changed from GUI (new) to Services
  • Assignee changed from Release Council to John Hixson
  • Target version changed from Backlog to 11.2-RC1

#6 Updated by Dru Lavigne about 2 years ago

  • Seen in changed from Unspecified to 11.2-BETA2

#7 Updated by John Hixson about 2 years ago

  • Assignee changed from John Hixson to Andrew Walker

#8 Updated by Andrew Walker almost 2 years ago

When we configure home directories in samba on freenas, we use the following path syntax

path = "/mnt/dozer/SMB/%U"

Note the %U. This is a macro that expands to a sanitized form of the session's username. One step of the sanitization process is to convert it to lower case.

Unfortunately, there's not much we can do to change this behavior without risking breaking things for current users (changing the macro to something like %H) or breaking samba (by changing sanitization methods for %U).

#9 Updated by Matt Bunce almost 2 years ago

To be honest, if there had been a warning about this on the field where you set the user home directory that would have been enough to save me a couple of hours of head scratching.

If it's clearly signposted it really isn't much of a problem.

#10 Updated by Andrew Walker almost 2 years ago

Matt Bunce wrote:

To be honest, if there had been a warning about this on the field where you set the user home directory that would have been enough to save me a couple of hours of head scratching.

If it's clearly signposted it really isn't much of a problem.

A warning sounds reasonable, but it should probably be during user account creation if the specified account name is uppercase or mixed-case. Lower-case usernames have been the convention for long enough it's better to not deviate.

#12 Updated by Dru Lavigne almost 2 years ago

  • Category changed from Services to Documentation
  • Assignee changed from Andrew Walker to Warren Block
  • Target version changed from 11.2-RC1 to 11.2-RC2
  • Reason for Blocked deleted (Need additional information from Author)

#14 Updated by Dru Lavigne almost 2 years ago

  • Related to Bug #51590: New user creation rules only lower case allowed added

#15 Updated by Warren Block almost 2 years ago

  • Subject changed from Selected user home dataset is no honored to Selected user home dataset is not honored
  • Assignee changed from Warren Block to Samantha Bonham

#16 Updated by Samantha Bonham almost 2 years ago

  • Status changed from Unscreened to In Progress

#17 Updated by Samantha Bonham almost 2 years ago

In response to option 2: Here is the PR for the tooltip, which makes a note not to use uppercase letters for usernames used for an SMB home share: https://github.com/freenas/webui/pull/1392

#18 Updated by Dru Lavigne almost 2 years ago

  • File deleted (debug.7z)

#19 Updated by Dru Lavigne almost 2 years ago

  • Subject changed from Selected user home dataset is not honored to Add tooltip warning users to avoid using uppercase usernames when creating an SMB home share
  • Private changed from Yes to No

#20 Updated by Samantha Bonham almost 2 years ago

#21 Updated by Dru Lavigne almost 2 years ago

  • Status changed from In Progress to Ready for Testing
  • Needs Merging changed from Yes to No

#23 Updated by Jeff Ervin almost 2 years ago

37320
37323

Test Failed FreeNAS-11.2-INTERNAL30 (30 created 10/29/18)

#24 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Failed Testing to In Progress

#25 Updated by Dru Lavigne almost 2 years ago

  • Status changed from In Progress to Failed Testing

#26 Updated by Jeff Ervin almost 2 years ago

37365

Test Passed FreeNAS-11.2-INTERNAL30

After speaking with Dru, this is where the tooltip is supposed to be.

#28 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Passed Testing to Done
  • Needs Doc changed from Yes to No

Also available in: Atom PDF