X2Go Bug report logs - #1230
x2go fails unhelpfully when a session fails to start

Full log

🔗 View this message in rfc822 format

X-Loop: owner@bugs.x2go.org
Subject: Bug#1230: X2Go issue (in src:x2goserver) has been marked as pending for release
Reply-To: Mihai Moldovan <ionic@ionic.de>, 1230-quiet@bugs.x2go.org
Resent-To: "Norman Gray" <gray@nxg.name>
X-Loop: owner@bugs.x2go.org
Resent-Date: Fri, 28 Feb 2020 14:40:06 +0000
Resent-Message-ID: <handler.1230.U1230.158290063323841@bugs.x2go.org>
Resent-Sender: owner@bugs.x2go.org
X-X2Go-PR-Message: report 1230
X-X2Go-PR-Package: x2goserver
X-X2Go-PR-Keywords: pending
References: <AEF0C8DA-B122-4D28-BBA4-3E3A6463C5DD@nxg.name>
Received: via spool by 1230-submitter@bugs.x2go.org id=U1230.158290063323841
          (code U ref 1230); Fri, 28 Feb 2020 14:40:06 +0000
Received: (at 1230-submitter) by bugs.x2go.org; 28 Feb 2020 14:37:13 +0000
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on
	ymir.das-netzwerkteam.de
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=3.0 tests=BAYES_00,NO_RELAYS,
	URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.2
Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005)
	id 86F3B5DD18; Fri, 28 Feb 2020 15:36:56 +0100 (CET)
From: Mihai Moldovan <ionic@ionic.de>
To: 1230-submitter@bugs.x2go.org
Cc: control@bugs.x2go.org, 1230@bugs.x2go.org
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
X-Mailer: https://x2go.org/post-receive-tag-pending
Message-Id: <20200228143656.86F3B5DD18@ymir.das-netzwerkteam.de>
Date: Fri, 28 Feb 2020 15:36:56 +0100 (CET)

tag #1230 pending
fixed #1230 4.0.1.23
thanks

Hi!

X2Go issue #1230 (src:x2goserver) reported by you has been
fixed in X2Go Git. You can see the changelog below, and you can
check the diff of the fix at:

    http://code.x2go.org/gitweb?p=x2goserver.git;a=commitdiff;h=7a76b291

The issue will most likely be fixed in src:x2goserver (4.0.1.23).

light+love
X2Go Git Admin (on behalf of the sender of this mail)

---
commit 7a76b291ed589775ee5d84c34b677fd621ce6d11
Author: Mihai Moldovan <ionic@ionic.de>
Date:   Thu Jan 4 06:16:51 2018 +0100

    x2goserver/bin/x2go{startagent,resume-session}: use x2gogetfreeport instead of duplicating the same code everywhere. Fixes: #1230.
    
    Also, rely on it returning a valid value and try at most 10 times to
    fetch a usable port value, then error out and make the session startup
    or resumption fail.

diff --git a/debian/changelog b/debian/changelog
index 818e823e..f8582d47 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -51,6 +51,11 @@ x2goserver (4.0.1.23-0x2go1) UNRELEASED; urgency=medium
       quotes on the lib path line.
     - x2goserver/bin/x2goresume-session: properly randomize first port, just
       like in x2gostartagent.
+    - x2goserver/bin/x2go{startagent,resume-session}: use x2gogetfreeport
+      instead of duplicating the same code everywhere. Fixes: #1230. Also,
+      rely on it returning a valid value and try at most 10 times to fetch a
+      usable port value, then error out and make the session startup or
+      resumption fail.
   * x2goserver.spec:
     - RPMify x2goserver-xsession description.
     - Remove qt4 stuff, we're not using the framework here.

Send a report that this bug log contains spam.