#50634 Clean up CLI errors output
Closed: wontfix 2 years ago by spichugi. Opened 2 years ago by spichugi.

Issue Description

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.

Also, python-ldap should be fixed so we process str(error) correctly.

Metadata Update from @spichugi:
- Custom field origin adjusted to None
- Custom field reviewstatus adjusted to None
- Issue tagged with: Easyfix

2 years ago

Metadata Update from @mreynolds:
- Issue set to the milestone: 1.4.0

2 years ago

Metadata Update from @spichugi:
- Issue assigned to spichugi

2 years ago

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.

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

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)

2 years ago

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.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: fixed)

2 years ago

Login to comment on this ticket.