X2Go Bug report logs - #1446
Domain name issues with client 4.1.2.2 when using HTTP broker

version graph

Package: x2goclient; Maintainer for x2goclient is X2Go Developers <x2go-dev@lists.x2go.org>; Source for x2goclient is src:x2goclient.

Reported by: Mark Hymers <mark.hymers@ynic.york.ac.uk>

Date: Thu, 19 Mar 2020 20:05:03 UTC

Severity: normal

Found in version 4.1.2.2

Reply or subscribe to this bug.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to x2go-dev@lists.x2go.org, X2Go Developers <x2go-dev@lists.x2go.org>:
Bug#1446; Package x2goclient. (Thu, 19 Mar 2020 20:05:03 GMT) (full text, mbox, link).


Acknowledgement sent to Mark Hymers <mark.hymers@ynic.york.ac.uk>:
New Bug report received and forwarded. Copy sent to X2Go Developers <x2go-dev@lists.x2go.org>. (Thu, 19 Mar 2020 20:05:03 GMT) (full text, mbox, link).


Message #5 received at submit@bugs.x2go.org (full text, mbox, reply):

From: Mark Hymers <mark.hymers@ynic.york.ac.uk>
To: submit@bugs.x2go.org
Subject: Domain name issues with client 4.1.2.2 when using HTTP broker
Date: Thu, 19 Mar 2020 20:03:00 +0000
[Message part 1 (text/plain, inline)]
Package: x2goclient
Version: 4.1.2.2


Hi,

We have an x2go deployment using the HTTP broker and multiple backend 
x2go servers.

We have found that if users use 4.1.2.2, when sessions are resumed the 
system tries to connect to the hostname without the domainname attached. 
  I.e. system1 instead of system1.example.org.  This is not a problem 
with 4.1.2.1 or earlier client releases (no server changes were made).

I have tracked this down to a change in session handling in 
resumeSession in src/httpbrokerclient.cpp.

In this function, config->serverIp is set to server which, at this point 
only contains the hostname, not the FQDN.  config->serverIp has already 
been set at this point by the parseSession function on line 700 (current 
git HEAD):

config->serverIp=words[0];

This version of config->serverIp is the proper FQDN.

To fix this, I just removed the overriding of serverIp in the 
resumeSession function (see attached patch) but I'm not sure if this is 
fully correct; presumably there was a reason for setting it in 
resumeSession.

Thanks,

Mark

-- 
Mark Hymers, Lecturer in Neuroimaging Methods
York NeuroImaging Centre and Department of Psychology
University of York, UK

Please note that my email address is: mark.hymers@ynic.york.ac.uk
Responses to messages sent to other addresses may be delayed

[x2goclient_fix_server_name.diff (text/x-patch, attachment)]

Send a report that this bug log contains spam.


X2Go Developers <owner@bugs.x2go.org>. Last modified: Tue Jul 7 00:44:17 2020; Machine Name: ymir.das-netzwerkteam.de

X2Go Bug tracking system

Debbugs is free software and licensed under the terms of the GNU Public License version 2. The current version can be obtained from https://bugs.debian.org/debbugs-source/.

Copyright © 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson, 2005-2017 Don Armstrong, and many other contributors.