CLI tools should print human easy readable messages if something went wrong. As discussed here: https://pagure.io/389-ds-base/pull-request/50624 we should format these error message Error: {'desc': 'Type or value exists'} without 'desc' and dict. It should be just Error: Type or value exists.
Error: {'desc': 'Type or value exists'}
Error: Type or value exists
Also, python-ldap should be fixed so we process str(error) correctly.
python-ldap
str(error)
Metadata Update from @spichugi: - Custom field origin adjusted to None - Custom field reviewstatus adjusted to None - Issue tagged with: Easyfix
Metadata Update from @mreynolds: - Issue set to the milestone: 1.4.0
Metadata Update from @spichugi: - Issue assigned to spichugi
https://pagure.io/389-ds-base/pull-request/50658
The upstream python-ldap issue is opened: https://github.com/python-ldap/python-ldap/issues/304
We should fix our code after it's resolved.
https://pagure.io/389-ds-base/pull-request/50663
9eba2ce..ea5eb49 389-ds-base-1.4.0 -> 389-ds-base-1.4.0 fb4ea1d..4e32c54 389-ds-base-1.4.1 -> 389-ds-base-1.4.1
https://pagure.io/389-ds-base/pull-request/50719
565d57b..994a109 389-ds-base-1.4.0 -> 389-ds-base-1.4.0 c0958e5..24ab95b 389-ds-base-1.4.1 -> 389-ds-base-1.4.1
Metadata Update from @spichugi: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/3689
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: fixed)
Login to comment on this ticket.