lucas-a-martins commented on PR #8603:
URL: https://github.com/apache/cloudstack/pull/8603#issuecomment-2091551541
> I know @lucas-a-martins
>
> but the new error message `Unable to complete this operation. Contact your
cloud admin.` is much more unclear
>
> ```
> LOGGER.error("Global setting [{}] cannot contain localhost or
be blank. Current value: {}", ApiServletPath.key(), csUrl);
> throw new InvalidParameterValueException("Unable to complete
this operation. Contact your cloud admin.");
> ```
@weizhouapache,
The exception being unclear is intentional, as this is what users will see.
I understand what you mean by showing an exclusive message to ROOT admins, but
is this really necessary since they have access to the logs with a clear
message? Especially considering that the current exception is unclear, so this
wouldn't change. I can't see a benefit in adding a "less unclear but not clear
enough" exception. Also, is there any case of an exception in the UI being
different between Users and Root Admins in the same situation?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
