Bug 39721 – mail/domain are not lower case

Bug 39721 - mail/domain are not lower case


Summary:	mail/domain are not lower case

Status:	CLOSED FIXED

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

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.1-0-errata
Assigned To:	Daniel Tröder
QA Contact:	Felix Botner

URL:
Keywords:

Depends on:
Blocks:	40108
	Show dependency tree / graph

Reported:	2015-11-03 08:57 CET by Daniel Tröder
Modified:	2015-12-09 16:46 CET (History)
CC List:	5 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
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-11-03 08:57:01 CET

Domain names for mail/domain should be lower cased.

According to RFC 1035, section 3.1: "Name servers and resolvers must compare labels in a case-insensitive manner" that would be allowed.

It would be consistent with what we do for the local part in the udm users handler (and probably solve a problem with shared folders and ACLs - still investigating).

Comment 1 Sönke Schwardt-Krummrich

2015-11-03 13:19:47 CET

This should also be fixed for UCS 4.0-4. Please also create a ucs-test script if possible.

Comment 2 Daniel Tröder

2015-11-18 18:50:34 CET

Code change for 4.1-0: 65733
Modified test for 4.1-0: 65732

Code + test change for 4.0-4: 65734

YAML 4.0-4 + 4.1-0 (r65735): 2015-11-18-univention-directory-manager-modules.yaml

Comment 3 Florian Best

2015-11-19 11:59:14 CET

Wouldn't it be better to solve this with the syntax class?

Comment 4 Daniel Tröder

2015-11-19 13:33:59 CET

Commit 65762 merges the YAML into the existing advisory 2015-11-17-univention-directory-manager-modules.yaml.

Comment 5 Sönke Schwardt-Krummrich

2015-11-19 14:24:32 CET

(In reply to Florian Best from comment #3)
> Wouldn't it be better to solve this with the syntax class?

Changing the syntax class wouldn't fix the existing LDAP objects. And fixing the LDAP objects without listeners would trigger the same (or worse) problems.

Comment 6 Philipp Hahn

2015-11-26 09:20:01 CET

<http://jenkins.knut.univention.de:8080/job/UCS-4.0/job/UCS-4.0-4/job/Errata%20Validation/lastCompletedBuild/testReport/univention-directory-manager-modules/bug/39721_scope_version/>

The same bug is referenced for both 4.0-4 and 4.1-0:

$ grep 39721 ucs-4.?/ucs-4.?-?/doc/errata/staging/*.yaml
ucs-4.0/ucs-4.0-4/doc/errata/staging/2015-11-18-univention-directory-manager-modules.yaml: * Domain names for mail domains are now lowercased (Bug 39721).
ucs-4.0/ucs-4.0-4/doc/errata/staging/2015-11-18-univention-directory-manager-modules.yaml:bug: [39721]
ucs-4.1/ucs-4.1-0/doc/errata/staging/2015-11-17-univention-directory-manager-modules.yaml:   behaviour with Dovecot and shared folders (Bug 39721).

Quoting Message-ID <564A3E28.9040607@univention.de>:
> Das bedeutet der Bug muss immer gklont werden, wenn dieser auch für UCS
> 4.0 veröffentlicht werden soll.

Comment 7 Daniel Tröder

2015-11-26 10:42:04 CET

Thanks.

Split 4.0-4 to Bug 40108.

Comment 8 Daniel Tröder

2015-11-26 10:42:15 CET

Thanks.

Split 4.0-4 to Bug 40108.

Comment 9 Felix Botner

2015-12-04 11:35:48 CET

OK - mail domain is lowercased
OK - 34_update_of_mail_hosteddomains_ucr_variable

OK - YAML

Comment 10 Janek Walkenhorst

2015-12-09 16:46:06 CET

<http://errata.software-univention.de/ucs/4.1/16.html>