Bug 35829 - Limit NETBIOS Domain Name to 15 characters
Limit NETBIOS Domain Name to 15 characters
Status: CLOSED WORKSFORME
Product: UCS
Classification: Unclassified
Component: System setup
UCS 4.2
Other Linux
: P5 normal (vote)
: UCS 4.3-1-errata
Assigned To: Richard Ulmer
Arvid Requate
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-04 16:04 CEST by Felix Botner
Modified: 2018-08-08 11:38 CEST (History)
5 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.023
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Botner univentionstaff 2014-09-04 16:04:22 CEST
At the moment there is no limit for the windows/domain (NETBIOS Domain Name) in system setup.
Comment 1 Alexander Kläser univentionstaff 2014-09-04 16:07:41 CEST
Currently, the domain name will be converted automatically to a NETBIOS compatible domain name (example-domain.local → EXAMPLE-DOMAIN). The limit of 15 characters is not checked/enforced at the moment.
Comment 2 Stefan Gohmann univentionstaff 2014-09-04 16:26:31 CEST
Do we need the NETBIOS domain during the S4 setup? If not, we could limit the NETBIOS domain. The value can later be changed via UCR. If we need it during the setup, we should re-add the field.
Comment 3 Stefan Gohmann univentionstaff 2017-06-16 20:38:01 CEST
This issue has been filed against UCS 3. UCS 3 is out of the normal maintenance and many UCS components have vastly changed in UCS 4.

If this issue is still valid, please change the version to a newer UCS version otherwise this issue will be automatically closed in the next weeks.
Comment 4 Richard Ulmer univentionstaff 2018-08-06 16:30:09 CEST
Enforcing a 15 character limit of the domain name now.

univention-system-setup (11.0.5-18)
954548ffebdd | Bug #35829: Limit domain name to 15 characters

univention-system-setup.yaml
eccbbbc5f258 | Bug #35829: Update yaml file
Comment 5 Richard Ulmer univentionstaff 2018-08-07 09:31:24 CEST
FYI: In the back end the 15 character limit is already enforced by now (see commit ec5ad3045e0c57c7bafb2652a51c4265a0f741e7 for Bug #31771 ).
Comment 6 Arvid Requate univentionstaff 2018-08-07 10:22:17 CEST
> FYI: In the back end the 15 character limit is already enforced by now (see commit ec5ad3045e0c57c7bafb2652a51c4265a0f741e7 for Bug #31771 ).

That's just an autopep8 commit with no functional change (hopefully).

git blame ec5ad3045e0c57c7bafb2652a51c4265a0f741e7^ base/univention-system-setup/umc/python/setup/util.py

shows f56260bed79 as origin of that line, which has been made for Bug #36459. So, as discussed via email, my impression is that we should not impose a limit on a part of the DNS domain name here that potentially disturbs customers that a) have longer domain names or b) even don't run Samba.
Comment 7 Richard Ulmer univentionstaff 2018-08-07 13:12:10 CEST
As discussed with Arvid I took the changes back. This bug was already solved in the scope of Bug #36459.

univention-system-setup.yaml
e95ceb21e077 | Bug #35829: Update yaml file

univention-system-setup (11.0.5-19)
d0bf4584a887 | Bug #35829: Revert "Bug #35829: Limit domain name to 15 characters"
Comment 8 Arvid Requate univentionstaff 2018-08-07 13:37:07 CEST
Ok:

* git diff  954548ffeb^ d0bf4584a8 base/univention-system-setup/usr/share/univention-system-setup/www/ApplianceWizard.js

* git diff 954548ffeb^ d0bf4584a8 base/univention-system-setup/usr/share/univention-system-setup/www/de.po

* git diff 954548ffeb^ e95ceb21e0 doc/errata/staging/univention-system-setup.yaml

* Binary version 11.0.5-19A~4.3.0.201808071304 matches source package version

* Binary version present in Advisory
Comment 9 Stefan Gohmann univentionstaff 2018-08-08 11:38:25 CEST
OK, nothing to release.