Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xpra control does not output error messages #2339

Closed
totaam opened this issue Jun 19, 2019 · 6 comments
Closed

xpra control does not output error messages #2339

totaam opened this issue Jun 19, 2019 · 6 comments
Labels

Comments

@totaam
Copy link
Collaborator

totaam commented Jun 19, 2019

Issue migrated from trac ticket # 2339

component: server | priority: minor | resolution: fixed

2019-06-19 10:41:00: stdedos created the issue


From #2338: Running:

$ xpra control 0 clipboard-direction help
$ 

Gives no output, but, in logs:


  72   │ 2019-06-19 12:22:45,821 error processing control command 'clipboard-direction'
  73   │ Traceback (most recent call last):
  74   │   File "/usr/lib/python2.7/dist-packages/xpra/server/server_core.py", line 672, in process_control_command
  75   │     v = command.run(*args[1:])
  76   │   File "/usr/lib/python2.7/dist-packages/xpra/server/control_command.py", line 69, in run
  77   │     return super(ArgsControlCommand, self).run(*args)
  78   │   File "/usr/lib/python2.7/dist-packages/xpra/server/control_command.py", line 31, in run
  79   │     return self.do_run(*args)
  80   │   File "/usr/lib/python2.7/dist-packages/xpra/server/mixins/server_base_controlcommands.py", line 452, in control_command_clipboard_direction
  81   │     assert direction in DIRECTIONS, "invalid direction '%s', must be one of %s" % (direction, csv(DIRECTIONS))
  82   │ AssertionError: invalid direction 'help', must be one of to-server, to-client, both, disabled
@totaam
Copy link
Collaborator Author

totaam commented Jun 19, 2019

2019-06-19 11:08:08: stdedos commented


I also got this from the client:

2019-06-19 12:24:05,877 unknown server setting changed: clipboard_direction="b'both'"

(I don't know if it helps with anything)

@totaam
Copy link
Collaborator Author

totaam commented Jun 19, 2019

2019-06-19 11:29:20: antoine changed owner from antoine to stdedos

@totaam
Copy link
Collaborator Author

totaam commented Jun 19, 2019

2019-06-19 11:29:20: antoine commented


I see:

$ xpra control :2 clipboard-direction help
server returned error code 127
 error processing control command: invalid direction 'help', must be one of to-server, to-client, both, disabled

Which is fine.
I originally changed it in r22995, but r22996 reverts it to ensure that we return a non-zero exit code when things fail.

Please provide more details so I can reproduce: commands used, exact version, exit code, etc..

@totaam
Copy link
Collaborator Author

totaam commented Jun 19, 2019

2019-06-19 12:16:13: stdedos commented


Server version is v3.0-r22963. So, I guess you can close as already fixed?

@totaam
Copy link
Collaborator Author

totaam commented Jul 3, 2019

2019-07-03 17:15:16: antoine changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Jul 3, 2019

2019-07-03 17:15:16: antoine set resolution to fixed

@totaam totaam closed this as completed Jul 3, 2019
@totaam totaam added the v2.5.x label Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant