feat(elektra/swift): the "Check ACLs" button returns confusing error when ACLs are not well formatted

sapcc / elektra

An opinionated openstack Web UI for consumer self service and operations.

Apache License 2.0

74 stars 29 forks source link

feat(elektra/swift): the "Check ACLs" button returns confusing error when ACLs are not well formatted #1366

Closed kayrus closed 3 months ago

kayrus commented 3 months ago

Type project1-uid: user1-id in both read and write ACLs (mind whitespace)

Type the Check ACLs button.

See

We are currently experiencing a higher latency in our backend calls. This should be fixed momentarily. Please try again in a couple of minutes.

error message.

Expected behavior: the error message must say that the ACLs are incorrect. Screenshot_20240612_152544

edda commented 3 months ago

This is our last resort catch-all for API 500 errors. We had planned to remove this message in favor of just stating that there was a 500 error but if this is indeed the case here, that this incorrect query causes a 500 on the backend then we can't really do much. Anyway we'll have a look what's happening here.

andypf commented 3 months ago

We should avoid catching 500 errors and displaying a generic error message. Instead, we should display the actual error message from the API

[ ] Remove the handling of 500 errors.
[ ] Test how the 500 errors are displayed in the UI.

edda commented 3 months ago

Will be fixed with #1374