Bug #3369

fetchmail delivery to AD users with capitalized names

Added by Filippo Carletti over 3 years ago. Updated over 3 years ago.

Status:CLOSEDStart date:
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:nethserver-fetchmail
Target version:v6.7
Security class: Resolution:
Affected version:v6.7 NEEDINFO:No

Description

If NethServer is joined to an Active Directory domain, trying to configure mail delivery from fetchmail to a user with a capital letter in the name fails with a validation error.
Steps to reproduce:
1. join NS to AD where username are like Filippo (capital initial)
2. go to fetchmail and configure delivery to Filippo
3. see the "reuiqred username" error

Associated revisions

Revision 28ccbcc7
Added by Giacomo Sanchietti over 3 years ago

Web UI: allow AD users with upper case letters. Refs #3369

History

#1 Updated by Filippo Carletti over 3 years ago

  • Status changed from NEW to TRIAGED
  • % Done changed from 0 to 20
  • Affected version set to v6.7

Modifying line 41 of /usr/share/nethesis/NethServer/Module/Fetchmail/Accounts/Modify.php like:

- array('account', Validate::USERNAME, \Nethgui\Controller\Table\Modify::FIELD),
+ array('account', Validate::ANYTHING, \Nethgui\Controller\Table\Modify::FIELD),

solves the problem.

I'm not sure that ANYTHING is a good validator.

#2 Updated by Giacomo Sanchietti over 3 years ago

  • Status changed from TRIAGED to ON_DEV
  • Assignee set to Giacomo Sanchietti
  • % Done changed from 20 to 30

#3 Updated by Giacomo Sanchietti over 3 years ago

  • Status changed from ON_DEV to MODIFIED
  • % Done changed from 30 to 60

Ugly but effective workaround: proposed patch has been merged.

#4 Updated by Giacomo Sanchietti over 3 years ago

  • Status changed from MODIFIED to ON_QA
  • Assignee deleted (Giacomo Sanchietti)
  • % Done changed from 60 to 70
In nethserver-testing:
  • nethserver-fetchmail-1.1.5-1.1.g28ccbcc.ns6.noarch.rpm
Test case
  • Check the bug is not reproducible

#5 Updated by Giacomo Sanchietti over 3 years ago

  • Assignee set to Giacomo Sanchietti

#6 Updated by Giacomo Sanchietti over 3 years ago

  • Status changed from ON_QA to VERIFIED
  • Assignee deleted (Giacomo Sanchietti)
  • % Done changed from 70 to 90

The validator correctly accepts user names with capitalized letters.

#7 Updated by Giacomo Sanchietti over 3 years ago

  • Status changed from VERIFIED to CLOSED
  • % Done changed from 90 to 100
Released in nethserver-updates:
  • nethserver-fetchmail-1.1.6-1.ns6.noarch.rpm

Also available in: Atom PDF