Bug 51783 - Make remove_s4_rejected.py non case-sensitive
Make remove_s4_rejected.py non case-sensitive
Status: NEW
Product: UCS
Classification: Unclassified
Component: S4 Connector
UCS 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: Samba maintainers
Samba maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2020-08-06 14:56 CEST by Christian Völker
Modified: 2020-08-06 14:56 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?: 2: Will only affect a 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.091
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2020080321000564
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christian Völker univentionstaff 2020-08-06 14:56:44 CEST
Troubleshooting is painful when running into this...
Happened while troubleshooting another issue.

===================
root@luiseedu:/var/log/univention# univention-s4connector-list-rejected 
UCS rejected


S4 rejected

    1:    S4 DN: CN=dns,DC=schulen,DC=ucs
         UCS DN: cn=dns,dc=schulen,dc=ucs

	last synced USN: 6602
=====================

BUT:
=====================
root@luiseedu:/var/log/univention# /usr/share/univention-s4-connector/remove_s4_rejected.py "cn=dns,dc=schulen,dc=ucs" 
ERROR: The object cn=dns,dc=schulen,dc=ucs was not found.
====================

Only using the capital letters made it work:
===================
root@luiseedu:/var/log/univention# /usr/share/univention-s4-connector/remove_s4_rejected.py "CN=dns,DC=schulen,DC=ucs"The rejected S4 object CN=dns,DC=schulen,DC=ucs has been removed.
===================

This is for sure not to be expected and should be handled accordingly.

At least improve the error message with something like: "This tool is case sensitive, please double-check DN!"