Bug 39346 – lowercase email addresses for Dovecot user queries

Bug 39346 - lowercase email addresses for Dovecot user queries


Summary:	lowercase email addresses for Dovecot user queries

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	Mail - Dovecot
Version:	UCS 4.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.0-3-errata
Assigned To:	Daniel Tröder
QA Contact:	Sönke Schwardt-Krummrich

URL:
Keywords:

Depends on:
Blocks:	39407
	Show dependency tree / graph

Reported:	2015-09-15 09:30 CEST by Daniel Tröder
Modified:	2015-09-25 13:01 CEST (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	---
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
r63689.diff (2.45 KB, patch) 2015-09-15 11:38 CEST, Stefan Gohmann	Details \| Diff
63756.diff (6.64 KB, patch) 2015-09-18 07:00 CEST, Stefan Gohmann	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Daniel Tröder

2015-09-15 09:30:05 CEST

A customer reports that an IMAP access with uppercase email address made Dovecot create a new mail home besides the existing one with lowercase directory name.

Although I could not reproduce this the change makes sense and should not hurt existing installations. I should furthermore raise the auth cache hit ratio.

The "%L" modifier changes all characters to lowercase when used in a userdb or passwd query.

Comment 1 Daniel Tröder

2015-09-15 09:48:58 CEST

Commit + merge to 4.1: 63689
YAML (r63690): 2015-09-07-univention-mail-dovecot.yaml

Comment 2 Stefan Gohmann

2015-09-15 11:38:30 CEST

Created attachment 7165 [details]
r63689.diff

Comment 3 Daniel Tröder

2015-09-16 14:02:21 CEST

Configuration and listener code adapted, so that aLL on-disk paths (except for shared folders w/o email address) are lowercased now too.

Code 4.0-3: 63756
Merge 4.1: 63757
YAML (63759): 2015-09-02-univention-mail-dovecot.yaml

Comment 4 Stefan Gohmann

2015-09-18 07:00:04 CEST

Created attachment 7171 [details]
63756.diff

Comment 5 Sönke Schwardt-Krummrich

2015-09-23 17:09:41 CEST

OK: code change
OK: functional test
    - see ucs-test script
    - adding/renaming shared folders with and without mailPrimaryAddress
ADDED: ucs-test/tests/40_mail/45_dovecot_test_camelcase_mailadresses
       - creates 4 users with lowercase, uppercase and mixed case mail addreses
       - checks if lower case mailbox is created (and not mixed/upper case)
       - mail is delivered to correct mailbox (→ lowercase)
       - if mailboxes are removed correctly after deleting user object
OK: YAML

Comment 6 Janek Walkenhorst

2015-09-24 14:37:35 CEST

<http://errata.software-univention.de/ucs/4.0/333.html>