xpra icon
Bug tracker and wiki

Opened 7 months ago

Closed 6 months ago

#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 (3)

comment:1 Changed 7 months ago by Antoine Martin

Component: androidserver
Milestone: 2.3
Priority: majorminor
Status: newassigned
Type: defectenhancement

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.

comment:2 Changed 7 months ago by Antoine Martin

Owner: changed from Antoine Martin to Boruch
Status: assignednew

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 6 months ago by Antoine Martin

Resolution: needinfo
Status: newclosed
Note: See TracTickets for help on using tickets.