Univention Bugzilla – Bug 31717
Passing additional parameters to RDP sessions not possible
Last modified: 2014-06-12 09:19:51 CEST
Created attachment 5273 [details] Fix RDP parameter handling It's not possible to pass additional parameters to an RDP session via UCR variable rdp/additionaloptions. In this case I tried to enable the background wallpaper by appending the "-x 0" parameter. Currently, there are two issue. First, parameters are append after the final "--" and are therefore treated as destination server. Second, the parameters are enclosed in quotes and therefore treated as one option which causes xfreerdp to throw an error message and quit for any combination other than a single switch like -z.
(In reply to Jan Christoph Ebersbach from comment #0) > parameters are append after the final "--" and are therefore treated as > destination server. Did you test this further (bash -x)? The actual call from the RDP session script is /usr/bin/xfreerdp "${params[@]}" $server while rdp/additionaloptions is part of the params array. The second issue is existent.
(In reply to Erik Damrose from comment #1) > (In reply to Jan Christoph Ebersbach from comment #0) > > parameters are append after the final "--" and are therefore treated as > > destination server. > Did you test this further (bash -x)? The actual call from the RDP session > script is > /usr/bin/xfreerdp "${params[@]}" $server > while rdp/additionaloptions is part of the params array. The second issue is > existent. Yes, I investigated the issue by sending the final command to logger and it turned out that the -x 0 parameter ended up behind --, right before the destination server. The session quit immediately. Even if the parameter would be moved before -- it wouldn't work, because the last plugin parameter treats every following parameter as directory. Only after moving params before the last plugin it finally worked.
I think the patch is only a workaround, not a proper fix. The handling of positional arguments in xfreerdp is quite tricky and was tested with several combinations and parameters. If there are argument, which require a different position, we might need a second UCR variable.
Before checking the patch the behaviour with the new freerdp should be tested.
The patch has been merged.
OK, works fine. Tested with -T test to set the rdp session window title
UCC 2.0 has been released: http://docs.univention.de/release-notes-ucc-2.0.html If this error occurs again, please use "Clone This Bug".