#1733 closed enhancement (needinfo)
Bad word-choice in error messages: FAILURE
Reported by: | Boruch | Owned by: | Boruch |
---|---|---|---|
Priority: | minor | Milestone: | 2.3 |
Component: | server | Version: | 2.2.x |
Keywords: | Cc: |
Description
The STDERR messages seem to be more alarmist than necessary, and give me a false feeling that xpra
itself may have failed, or that some critical feature has failed, when such does not seem to be the case. The following sample paste of an xpra
STDERR illustrates with two examples:
1511 $2018-01-03 00:02:08,828 server failure: disconnected before the session could be established 2018-01-03 00:02:08,830 server requested disconnect: server error (error accepting new connection) 2018-01-03 00:02:08,846 server failure: disconnected before the session could be established 2018-01-03 00:02:08,847 server requested disconnect: server error (error accepting new connection) Warning: cannot use the system proxy for 'start' subcommand, FAILURE
1] The initial messages indicate that the server experienced a failure, but the server seems fine.
2] The final message issues a warning, followed by a cryptic (and alarming) "FAILURE".
This choice of words does xpra
a dis-service, in that it makes me (and probably other users) unsure and uncomfortable with the product's stability and functionality.
My suggestion is to reserve the word 'failure' for situations that render the software unusable. Maybe otherwise use the word "error". Alternatively, a term such as "non-critical failure" might be appropriate.
Change History (4)
comment:1 Changed 4 years ago by
Component: | android → server |
---|---|
Milestone: | → 2.3 |
Priority: | major → minor |
Status: | new → assigned |
Type: | defect → enhancement |
comment:2 Changed 4 years ago by
Owner: | changed from Antoine Martin to Boruch |
---|---|
Status: | assigned → new |
I cannot reproduce the warning message you are seeing:
- if the system socket exists but cannot be used, you get:
Warning: cannot use the system proxy for 'start' subcommand, failed to connect to '/run/xpra/system': [Errno 111] Connection refused
With more details in debug mode. This is fine.
- if I point it to a location that does not exist (r17838 allows us to override the path to make it easier to test - here with
/tmp/hello
):Warning: cannot use the system proxy for 'start' subcommand, failed to connect to '/tmp/hello': [Errno 2] No such file or directory
And this also looks fine to me.
So, there must be something different going on with your system.
r17839 changes the wording to "unknown general failure" because that's all I know about your specific failure state.
comment:3 Changed 4 years ago by
Resolution: | → needinfo |
---|---|
Status: | new → closed |
comment:4 Changed 16 months ago by
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1733
This warning message should only be shown when the system service is not available, this should not happen with the packages we provide.
The "FAILURE" message is a generic lookup of the sub-command's exit code, I'll try to special case it to make it more user friendly.