From X2Go-ML-1@baur-itcs.de Tue Mar 22 10:54:55 2016 Received: (at submit) by bugs.x2go.org; 22 Mar 2016 09:54:56 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=3.0 tests=BAYES_00 autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 1E6825DAB9 for ; Tue, 22 Mar 2016 10:54:55 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8gkpw6TGMqGM for ; Tue, 22 Mar 2016 10:54:49 +0100 (CET) Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.126.134]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id F226D5DA92 for ; Tue, 22 Mar 2016 10:54:48 +0100 (CET) Received: from [192.168.0.171] ([109.193.81.164]) by mrelayeu.kundenserver.de (mreue004) with ESMTPSA (Nemesis) id 0MbtgI-1aSQ2A2Wms-00JKne for ; Tue, 22 Mar 2016 10:54:48 +0100 To: submit@bugs.x2go.org From: Stefan Baur Subject: x2gobroker-daemon doesn't return error when it can't spawn/bind to port Message-ID: <56F11663.5060902@baur-itcs.de> Date: Tue, 22 Mar 2016 10:54:43 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.0 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="UHwSxnslJLA1p401MB9OJLQphx9EKwqI6" X-Provags-ID: V03:K0:CeQID6aH+z2XoTfiFMmwDQiVNhXNR4PMrMuEGomYySrh8BZU+Xw pfIT+qC00CE6AY218fCU6VNGIT+zC4RhlLoyrRYHfpmbe4E09z3AHgj/3NMFgkcT0OLxsuJ N6aefaGtJ2ddmmF5HmqAJF+8XD7mkF5ihyrb/gE2jEhlNpg6jEscWMC/MV3GWdEDg4HqJ1A K0+8PtH0xrKqp2eB8OyOA== X-UI-Out-Filterresults: notjunk:1;V01:K0:C7uEFLZO5Wc=:eDnnU4Qjh8fVkpIQRy8ItO KW2qflIzntpSds67R4FCKFmsdbCUM3r4bsDrdLpUn6j5jnhGnfxc/f0sW2Mzm3i2N9ndEK5xG 3TPWbAhk7H+4LMtXiHwzFvNDq3z8JbPMLE/bBNzYfANoFFwjzJ0YBOvfyg4/jrDWuVoIAqsJo XHoivS4k8Z4Q6znFPLQnmzoKObY4uSK5pe2Xqqs3QnMCBGgc70gRsVx3bGv6cwvu4qpp4K3C8 JYlGRWD2HLUa9ligulrX2U0UJmw2QH24gNWdlPaWmA76uxIUKiKIsoCda+AzFY8RhIPwizxD/ 6HSJ1XZiKXbh6hNZ83lBui4Vurdbgnajo3BCmApNugnfmJ5BWvqjsY6vHsWTWwDmrPFyKbLbq VT43QM7IN9W3mNt2qMuphbA+jfieK079JJLdTSdmG+w/6pFmfjTMetVk1qrPq78DsDAiMTYsP PswBMUhOwgAUXK8TP4xD8LWgHEpyZaZ1HtThTJ+5WoUwkFTnja+3eqXU/zJukp9VKpTJv0zT+ oa1sXFOtILTjoZtDQFS9azofX9Oi8b+mr+tBPq+IP7ue24OYa03tyG6mFoVsg3dViOU5glpgr SnrQy2XpqBywlRW25ABfKgrbW2VSVQ9nQ/7yxfd0Xtk+zcVlgINO8PElS4PPPQjYEeD0FjHwz 8Sbg4xzrY+XOpyg5raCWC9/d6bmAVliWfTe5wlS3rb8iBCI2UanZDyhume+FadvJOefQ= This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --UHwSxnslJLA1p401MB9OJLQphx9EKwqI6 Content-Type: multipart/mixed; boundary="OoHjjxj9T4Bp3wDBh959PMn0P1UxXXd6g" From: Stefan Baur To: submit@bugs.x2go.org Message-ID: <56F11663.5060902@baur-itcs.de> Subject: x2gobroker-daemon doesn't return error when it can't spawn/bind to port --OoHjjxj9T4Bp3wDBh959PMn0P1UxXXd6g Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable package: x2gobroker-daemon version: 0.0.3.0 This is what happened: node:~# service x2gobroker-daemon stop [....] Stopping X2Go Session Broker standalone daemon: x2gobroker-daemonstart-stop-daemon: warning: failed to kill 14170: No such process failed! # huh, wait, what? node:~# ps ax | grep brok 5499 ? S 0:00 x2gobroker-authservice -s /run/x2gobroker/x2gobroker-authservice.socket -o root -g x2gobroker -p 06= 60 18791 pts/0 S+ 0:00 grep brok # no running x2gobroker-daemon, so ... # let's remove the PID file: node:~# rm /run/x2gobroker/x2gobroker-daemon.pid node:~# service x2gobroker-daemon stop node:~# service x2gobroker-daemon start [ ok ] Starting X2Go Session Broker standalone daemon: x2gobroker-daemon.= # looking good so far ... node:~# service x2gobroker-daemon stop [....] Stopping X2Go Session Broker standalone daemon: x2gobroker-daemonstart-stop-daemon: warning: failed to kill 18470: No such process failed! # whoops, again? node:~# ps ax | grep brok 5499 ? S 0:00 x2gobroker-authservice -s /run/x2gobroker/x2gobroker-authservice.socket -o root -g x2gobroker -p 06= 60 18791 pts/0 S+ 0:00 grep brok # no running x2gobroker-daemon, so ... # let's remove the PID file: node:~# rm /run/x2gobroker/x2gobroker-daemon.pid # I wonder what a restart will do? node:~# service x2gobroker-daemon restart [ ok ] Starting X2Go Session Broker standalone daemon: x2gobroker-daemon.= node:~# service x2gobroker-daemon restart [....] Stopping X2Go Session Broker standalone daemon: x2gobroker-daemonstart-stop-daemon: warning: failed to kill 18641: No such process failed! [warn] X2Go Session Broker: stale PID file (/run/x2gobroker/x2gobroker-daemon.pid). Delete it manually! ... (warning= ). node:~# ps ax | grep brok 5499 ? S 0:00 x2gobroker-authservice -s /run/x2gobroker/x2gobroker-authservice.socket -o root -g x2gobroker -p 06= 60 18791 pts/0 S+ 0:00 grep brok # and again, no running x2gobroker-daemon :-( The reason, as I found out later, was, that it tries to bind to 127.0.0.1:8080 by default, and that port is already in use by another process on that machine. IMO, this is a bug: when it can't bind to the port, and thus dies, this should be returned to the start script, so it can report back "FAILED". -Stefan --=20 BAUR-ITCS UG (haftungsbeschr=C3=A4nkt) Gesch=C3=A4ftsf=C3=BChrer: Stefan Baur Eichen=C3=A4ckerweg 10, 89081 Ulm | Registergericht Ulm, HRB 724364 Fon/Fax 0731 40 34 66-36/-35 | USt-IdNr.: DE268653243 --OoHjjxj9T4Bp3wDBh959PMn0P1UxXXd6g-- --UHwSxnslJLA1p401MB9OJLQphx9EKwqI6 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJW8RZnAAoJEG7d9BjNvlEZS00IAKvdn7OfKs2xzQ7mU14c48h3 KJbdvtFpnTcZ6MVjQ+J7YLmJHgOE0S9q36xz8gVMGz42r1gEO1SyWEX1AMhEbjNY DCNzOk1YrKlVsXdH8zLvVidzSazZIN5TsPhXUSP03p/Av1DFnkOoDuKnFrNTTMa3 uHBj7vvsYVMvvie1E+d/43/gX9Vglltncig0G/VE7brj91/kqurF9VpBetQWg3iU F4rqU3tpI+g+DZeWJPoMBhiNApHIu2DQn2f4ntLbbd/9PtT//kUOkuBt43qJQ2VT vjuW/Oj7FVwyX9mTNvCWh4qd7yCGyrSFTWL8G7tKGNek3WHgmUHaiUZHoTVtljA= =W9jw -----END PGP SIGNATURE----- --UHwSxnslJLA1p401MB9OJLQphx9EKwqI6-- From mike.gabriel@das-netzwerkteam.de Tue Mar 22 12:01:57 2016 Received: (at 1013) by bugs.x2go.org; 22 Mar 2016 11:01:58 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=3.0 tests=BAYES_00,URIBL_BLOCKED autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 42C665DAC3 for <1013@bugs.x2go.org>; Tue, 22 Mar 2016 12:01:57 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Jx4Q2olcTZDT for <1013@bugs.x2go.org>; Tue, 22 Mar 2016 12:01:50 +0100 (CET) Received: from freya.das-netzwerkteam.de (freya.das-netzwerkteam.de [88.198.48.199]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id 785405DA92 for <1013@bugs.x2go.org>; Tue, 22 Mar 2016 12:01:50 +0100 (CET) Received: from grimnir.das-netzwerkteam.de (grimnir.das-netzwerkteam.de [78.46.204.98]) by freya.das-netzwerkteam.de (Postfix) with ESMTPS id 514492A3; Tue, 22 Mar 2016 12:01:50 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by grimnir.das-netzwerkteam.de (Postfix) with ESMTP id F38073BA36; Tue, 22 Mar 2016 12:01:49 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at grimnir.das-netzwerkteam.de Received: from grimnir.das-netzwerkteam.de ([127.0.0.1]) by localhost (grimnir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id UBZfvRpt-rSO; Tue, 22 Mar 2016 12:01:43 +0100 (CET) Received: from das-netzwerkteam.de (localhost [127.0.0.1]) by grimnir.das-netzwerkteam.de (Postfix) with ESMTPS id 5613A3C167; Tue, 22 Mar 2016 12:01:34 +0100 (CET) Received: from 213.158.96.150 ([213.158.96.150]) by mail.das-netzwerkteam.de (Horde Framework) with HTTP; Tue, 22 Mar 2016 11:01:34 +0000 Date: Tue, 22 Mar 2016 11:01:34 +0000 Message-ID: <20160322110134.Horde.sDehV0sfKjEVIRRj82C9dAI@mail.das-netzwerkteam.de> From: Mike Gabriel To: Stefan Baur , 1013@bugs.x2go.org Subject: Re: [X2Go-Dev] Bug#1013: x2gobroker-daemon doesn't return error when it can't spawn/bind to port In-Reply-To: <56F11663.5060902@baur-itcs.de> User-Agent: Horde Application Framework 5 Accept-Language: de,en Organization: DAS-NETZWERKTEAM X-Originating-IP: 213.158.96.150 X-Remote-Browser: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Iceweasel/38.6.1 Content-Type: multipart/signed; boundary="=_GgZiAOTrjuX00it4_fF8dLk"; protocol="application/pgp-signature"; micalg=pgp-sha256 MIME-Version: 1.0 This message is in MIME format and has been PGP signed. --=_GgZiAOTrjuX00it4_fF8dLk Content-Type: text/plain; charset=utf-8; format=flowed; DelSp=Yes Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Control: severity -1 minor HI Stefan, On Di 22 M=C3=A4r 2016 10:54:43 CET, Stefan Baur wrote: > The reason, as I found out later, was, that it tries to bind to > 127.0.0.1:8080 by default, and that port is already in use by another > process on that machine. > > IMO, this is a bug: when it can't bind to the port, and thus dies, this > should be returned to the start script, so it can report back "FAILED". Yeah, this is a valid issue. Is you system launching x2gobroker-daemon=20= =20 via=20systemd or via System V init scripts? Mike --=20 DAS-NETZWERKTEAM mike=20gabriel, herweg 7, 24357 fleckeby fon: +49 (1520) 1976 148 GnuPG Key ID 0x25771B31 mail: mike.gabriel@das-netzwerkteam.de, http://das-netzwerkteam.de freeBusy: https://mail.das-netzwerkteam.de/mailxchange/kronolith/fb.php?u=3Dm.gabriel= %40das-netzwerkteam.de --=_GgZiAOTrjuX00it4_fF8dLk Content-Type: application/pgp-signature Content-Description: Digitale PGP-Signatur Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJW8SYOAAoJEJr0azAldxsx+SAQALIZ1WQWnqMloDlE8YBot4Nv T3hBUVIQ5iebS4GEsr39AvniVWs3byZbu2VDuZyMKytE31yW0/eJJEgu09jMraXi Kt+/orwY+2LTh8WGZTPiX8OECpcJMBzU/14ZexZ8+UbHO9jPqy8hiDtQKfMPAIAV bGQi/KTxIncuEzfwhUTNO+SO4TIHUMl1byx2DKQ+WdIhVHw7PYjHbve7UUpjbn0O /96J7uRlQuc7yo3oCELKvnGc2QS5lPI7gzh2iRrXshL89Qt9l2p3pRHTxz2bMfdf m+dEMclrwAklFK+S3h8e44hFqIbSl2I0mRw8DP5CxraJfUXuy+jKyAo77oqfwILr JjSrRgf0ZJk27V/grz3J0IcvwvV3rc/nKFqEhNDn7dElCWZVNih07/UkRr88PTt7 FFQIPtAs1S79DtRc16VnVg0QybV3gth+71uUsu6jJm1t4PkAYOfQ/IK2eUvfCP7T 09h61ssIVsx5rF4Qs5gPkcn/Gv7EixRx7qugBnAyGcEJM669Hxl2bBNTk7KOyLbE sPbABBPmAjhwNTegcl0zrJLh1+0oRJfQ3DCQL6gclYAwGVaWO3P0FTtT++S/nH1C ANoYnI0hX0VZc86D9bKkWBduVZek5ALJITqSJUKf+KKSk7JBGa89ylbZ0ApdDmZO QPHoiILIQbGaIAcHLEZg =nSyI -----END PGP SIGNATURE----- --=_GgZiAOTrjuX00it4_fF8dLk-- From X2Go-ML-1@baur-itcs.de Tue Mar 22 12:09:02 2016 Received: (at 1013) by bugs.x2go.org; 22 Mar 2016 11:09:04 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=3.0 tests=BAYES_00 autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id A1DF25DAB9 for <1013@bugs.x2go.org>; Tue, 22 Mar 2016 12:09:02 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id bzPsvO0k7F4U for <1013@bugs.x2go.org>; Tue, 22 Mar 2016 12:08:56 +0100 (CET) X-Greylist: delayed 301 seconds by postgrey-1.34 at ymir.das-netzwerkteam.de; Tue, 22 Mar 2016 12:08:56 CET Received: from mout.kundenserver.de (mout.kundenserver.de [217.72.192.74]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id 74FF05DA92 for <1013@bugs.x2go.org>; Tue, 22 Mar 2016 12:08:56 +0100 (CET) Received: from [192.168.0.171] ([109.193.81.164]) by mrelayeu.kundenserver.de (mreue103) with ESMTPSA (Nemesis) id 0MCx2B-1aZlOb0OFX-009gDU; Tue, 22 Mar 2016 12:03:54 +0100 Subject: Re: [X2Go-Dev] Bug#1013: x2gobroker-daemon doesn't return error when it can't spawn/bind to port To: Mike Gabriel , 1013@bugs.x2go.org References: <20160322110134.Horde.sDehV0sfKjEVIRRj82C9dAI@mail.das-netzwerkteam.de> From: Stefan Baur Message-ID: <56F12698.5040002@baur-itcs.de> Date: Tue, 22 Mar 2016 12:03:52 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.0 MIME-Version: 1.0 In-Reply-To: <20160322110134.Horde.sDehV0sfKjEVIRRj82C9dAI@mail.das-netzwerkteam.de> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="QqeteUOqoif283GMrWOfaCnKTkJJB971f" X-Provags-ID: V03:K0:DVUeYw3rLTz9KPrVcLbosunZSMlCODTHtXdSgyNcnjXdwovBdKB DIOzU8qKO/3ezlPdoz9vug3dRpG9hAKuTYLV28kI37xSOg9AaHWXS3ji/PLnUnl83OStix6 60iwqLOOFUlKdfuWLHB1zU2fd5Rbfo/V/lv986+RkU5pjcC2WgbgwHkIhx64+X14XMURWLK c356VXKOJ5QkYd5bJXnvA== X-UI-Out-Filterresults: notjunk:1;V01:K0:mZFSEUPh528=:mV83ulXB5O4GRbXde4ewVC aI/DTNretXO3hKaeLt1oQHuQsCGXYq7TAP1/qkqSCwEYkpGIGc1a99z7E0f40KHlQKiqwtGof Z2lsFMoFjmWwxWq8gacicovIRGIbJRm/s91tl33lH6Xsf+0T4PXHko3RDCae2YlbQNwV4jy/p LtBcGJ6AJHfJr0VZoP5neJkwJoEcc/7QPVdQrcmAPiridzPUzGkEJw6gRi4K52rM5qi5lbrci sLUT/4MVZA2WpQI+DP7gMg6Rel3RtxRY9RPj1/eBkSvHCa2q8UaAuZjRzUxDd70R5WVwszy53 gScGHKCr8Cp0w6je1Y7dpL/m5+OAQiSkE1pNpSPlT3k0WUIlQbnxjux38djI4463KCbFpjcwA toeehzqnI5YdpMSYHwlHCzryJm0U+AaLOWODyXGUEuZzsYWf/gOgDKLb4eFn/gd6yb+Eb6XwU bZ6uygHDlu80CUCsKIgfZ5ahFXK67kQD24VjGidVBGj27iwhK9UrRANcKUYRlLgK97ihntZhj GnFA8wvWVNlQmtCa8+isGrUbZcb6IMOIK4mE46LoLg8wCSC7jlUAB6/8M4TcvJWum8w1BYHB8 9t7ZXIPJR/lccZrsWVG01DXjNvpE00uRPLusmEup/LlH8HVqzGSDK4/fOEt9bTSP/zXdBvKYJ ZBfIH59SQJZS1+O2soresLUckTHOhKHKqdPgfQSQMAVvSWTgIFFZmvgpkpPTFoIOlGS0= This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --QqeteUOqoif283GMrWOfaCnKTkJJB971f Content-Type: multipart/mixed; boundary="cSMuXDHIstRdHk9V6rmCLSaHucHUVr7ju" From: Stefan Baur To: Mike Gabriel , 1013@bugs.x2go.org Message-ID: <56F12698.5040002@baur-itcs.de> Subject: Re: [X2Go-Dev] Bug#1013: x2gobroker-daemon doesn't return error when it can't spawn/bind to port References: <20160322110134.Horde.sDehV0sfKjEVIRRj82C9dAI@mail.das-netzwerkteam.de> In-Reply-To: <20160322110134.Horde.sDehV0sfKjEVIRRj82C9dAI@mail.das-netzwerkteam.de> --cSMuXDHIstRdHk9V6rmCLSaHucHUVr7ju Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Am 22.03.2016 um 12:01 schrieb Mike Gabriel: >=20 > Yeah, this is a valid issue. Is you system launching x2gobroker-daemon > via systemd or via System V init scripts? This is on System V init, as we've seen SystemD misbehaving with some of our own init scripts. Kind Regards, Stefan --=20 BAUR-ITCS UG (haftungsbeschr=C3=A4nkt) Gesch=C3=A4ftsf=C3=BChrer: Stefan Baur Eichen=C3=A4ckerweg 10, 89081 Ulm | Registergericht Ulm, HRB 724364 Fon/Fax 0731 40 34 66-36/-35 | USt-IdNr.: DE268653243 --cSMuXDHIstRdHk9V6rmCLSaHucHUVr7ju-- --QqeteUOqoif283GMrWOfaCnKTkJJB971f Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJW8SaYAAoJEG7d9BjNvlEZqF4H/1u7IGGSXVRoPxv4JCCjH1zN oI0JUDJ83BcNnDNlmWfDGyQb44Wl8Hc+/bD9sBmMSjvWfW38h3Xbra6W/pbAitbb E1SF/Ld8CfVi7SDvwIj2hIkiwNvyfASG0OHtWYw66n/GM6YW6/GBVbUJ88ShnC8I lg+wv7un3EPseOBOeDaG2+kUxvytnD8FoCcilvWikoSpRzoNSsTq8t/3C4mAUbZ7 oetgEzFxBLdv/3WzXGLQexUVQXH5foTtI/AhFvJViQjNj6wCnJ0vFxegzo0tt0oM KQKw9dnee+sHYawCIMd6bj5YqxTtLeSFhyI7xtAeA2KZFhg9Ey0RfS1ncaqITWY= =ekCT -----END PGP SIGNATURE----- --QqeteUOqoif283GMrWOfaCnKTkJJB971f-- From x2go@ymir.das-netzwerkteam.de Thu Sep 13 15:26:35 2018 Received: (at 1013) by bugs.x2go.org; 13 Sep 2018 13:26:45 +0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) 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.1 Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 3812D5DAEE; Thu, 13 Sep 2018 15:26:35 +0200 (CEST) From: =?utf-8?q?Mike_Gabriel?= To: 1013-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 1013@bugs.x2go.org Subject: X2Go issue (in src:x2gobroker) has been marked as pending for release Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit X-Mailer: http://snipr.com/post-receive-tag-pending Message-Id: <20180913132635.3812D5DAEE@ymir.das-netzwerkteam.de> Date: Thu, 13 Sep 2018 15:26:35 +0200 (CEST) tag #1013 pending fixed #1013 0.0.4.0 thanks Hello, X2Go issue #1013 (src:x2gobroker) 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=x2gobroker.git;a=commitdiff;h=c3067f0 The issue will most likely be fixed in src:x2gobroker (0.0.4.0). light+love X2Go Git Admin (on behalf of the sender of this mail) --- commit c3067f00df60d43ebbbecf5dd6f0f8da3c4221e4 Author: Mike Gabriel Date: Thu Sep 13 15:26:01 2018 +0200 bin/x2gobroker: If binding the http server fails, a non-zero exit code should be returned. (Fixes: #1013). diff --git a/debian/changelog b/debian/changelog index fc33189..7217c4f 100644 --- a/debian/changelog +++ b/debian/changelog @@ -75,6 +75,8 @@ x2gobroker (0.0.4.0-0x2go1) UNRELEASED; urgency=medium install into x2gobroker bin:pkg (on DEB based systems). - Makefile.docupload: Add apidoc target (running sphinx-apidoc). - docs/source: Initialize Sphinx API documentation's .rst files. + - bin/x2gobroker: If binding the http server fails, a non-zero exit code + should be returned. (Fixes: #1013). * debian/*: + Trigger Makefile's install target and install those files. Drop debhelper from-source-installation magic. From x2go@ymir.das-netzwerkteam.de Sat Feb 2 22:08:26 2019 Received: (at 1013) by bugs.x2go.org; 2 Feb 2019 21:08:47 +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=ham autolearn_force=no version=3.4.2 Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 4CF6B5DAED; Sat, 2 Feb 2019 22:08:25 +0100 (CET) From: =?utf-8?q?X2Go_Release_Manager?= X2Go Release Manager To: 1013-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 1013@bugs.x2go.org Subject: X2Go issue (in src:x2gobroker) has been marked as closed Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Message-Id: <20190202210825.4CF6B5DAED@ymir.das-netzwerkteam.de> Date: Sat, 2 Feb 2019 22:08:25 +0100 (CET) close #1013 thanks Hello, we are very hopeful that X2Go issue #1013 reported by you has been resolved in the new release (0.0.4.0) of the X2Go source project »src:x2gobroker«. You can view the complete changelog entry of src:x2gobroker (0.0.4.0) below, and you can use the following link to view all the code changes between this and the last release of src:x2gobroker. http://code.x2go.org/gitweb?p=x2gobroker.git;a=commitdiff;h=a2455880e34e31546054ce50abd1512c61430b51;hp=dbea0c7c20c58e6783ea796691f0881131ad6590 If you feel that the issue has not been resolved satisfyingly, feel free to reopen this bug report or submit a follow-up report with further observations described based on the new released version of src:x2gobroker. Thanks a lot for contributing to X2Go!!! light+love X2Go Git Admin (on behalf of the sender of this mail) --- X2Go Component: src:x2gobroker Version: 0.0.4.0-0x2go1 Status: RELEASE Date: Sat, 02 Feb 2019 21:50:29 +0100 Fixes: 1013 1234 1240 1252 1315 Changes: x2gobroker (0.0.4.0-0x2go1) RELEASED; urgency=medium . [ Mike Gabriel ] * New upstream version (0.0.4.0): - Bump upstream version to 0.0.4.0. - Port to Python 3. (Fixes: #1240). - Drop left-over debug print() call. - Makefile: Assure that setup.py is run under Python3. - Improve debugging messages during authentication phase. - x2gobroker/basicauth.py: Fix call of base64.decodestring on Python3. - Unit tests: Fix deep misunderstanding in the way allow-deny vs. deny-allow should actually work. - x2gobroker/brokers/base_broker.py: Entire rewrite of check_profile_acls() method. (Fixes: #1234). - x2gobroker/tests/test_web_plain_base.py: Add test case for passwords with accentuated characters (using the testsuite_authmech for now). - Makefile: Support skipping installation of the x2gobroker PyModule. Useful when building with CDBS on Debian. - Makefile: Compress man pages. - Makefile: Run setup.py build at build time. - tmpfiles.d utilization: Create RUNDIR/x2gobroker via tmpfiles.d system. Fixes missing dir and flawed permissions when running under systemd. - etc/x2gobroker.conf: Mention the per-profile option for enabling/disabling load checker support. - sbin/{x2gobroker-pubkeyauthorizer,x2gobroker-keygen}: Use proper octal numbers for file permissions. - sbin/x2gobroker-pubkeyauthorizer: Fix key lookup in os.environ for Python3. - sbin/x2gobroker-pubkeyauthorizer: Some string/bytecode fixes for Python3. Plus urllib -> urllib.request. - sbin/x2gobroker-pubkeyauthorizer: Improve key integrity checker and move it further up. Plus one more Python2 -> Python3 issue fixed. - sbin/x2gobroker-pubkeyauthorizer: Drop unused binascii import. - x2gobroker-pubkeyauthorizer: Tiny Python2to3 fix. - load checker integration: Make the default-use-load-checker option work like all other default-* options. - uccs frontend: Convert datetime.datetime object to string before answering the http request with it. - x2gobroker/agent (check_load()): Bail out if no remote agent is given. - x2gobroker-testagent: Convert to Python3 (using 2to3 tool). - x2gobroker-loadchecker: Python3'ify iteration over dict keys. - x2gobroker/utils.py: Provide helper functions for pretty-formatting key fingerprints. - x2gobroker-keygen: Use new fingerprint formatting functions. - x2gobroker/agent.py: Bail out if no hostaddr contained in remote_agent. - x2gobroker/agent.py: No load-checking when remote_agent is set to 'LOCAL'. - x2gobroker/agent.py: Better sanity checks for remote_agent and its dict keys hostname and hostaddr. - x2gobroker/loadchecker.py: Report properly to the logger if we fail to obtain a load factor. - x2gobroker-loadchecker.service: loadchecker service needs to chuid to system user x2gobroker. (Fixes: #1252). - x2gobroker-loadchecker.service: File ownership should be x2gobroker:x2gobroker, too. - x2gobroker-loadchecker: No chown/chmod if we are not running as root (which is mostly the case). - x2gobroker/brokers/inifile_broker.py: Make sure profile['name'] has a fallback if not given in the session profile. - x2gobroker/brokers/inifile_broker.py: Also check for presence of 'host' and 'sshport'. - UCCS API change for X2Go Sessions: Rename "SessionType" to "Command". - obligatory profile keys: Move from inifile backend to UCCS frontend, as those requirements are frontend specific. - UCCS: Start working on API version 5. - x2gobroker/uccsjson.py: Hide private Python class properties from JSON dict (like ._api_version). - UCCS frontend: Fix API version check. - UCSS frontend: Propagate API version onwards to the X2GoServer JSON generator class. - infile broker backend: Fix handling of empty lists in session profile and session profile defaults. - etc/x2gobroker-wsgi.apache.*: Drop Apache2.2 support. - Log to system broker.log file when run via x2gobroker-ssh. - Getting started documentation: Rework document, convert to markdown, install into x2gobroker bin:pkg (on DEB based systems). - Makefile.docupload: Add apidoc target (running sphinx-apidoc). - docs/source: Initialize Sphinx API documentation's .rst files. - bin/x2gobroker: If binding the http server fails, a non-zero exit code should be returned. (Fixes: #1013). - x2gobroker/loadchecker.py: Don't re-read the x2gobroker.conf during each cycle of the load checking loop. Rather read it on service startup and require a service restart when x2gobroker.conf has been changed. - x2gobroker/loadchecker.py: Avoid rare cases where at the end of a load checking cycle a negative sleep time would have been calculated. (Fixes: #1315). Thanks to Walid Moghrabi for catching this. - HTTP broker: Add &login= support to plain and json broker frontends. - SSH broker: Add --login option. This now supports X2Go Broker user and X2Go Server username being different accounts. - bin/x2gobroker: Correctly use split_host_address() function call. - bin/x2gobroker: Don't override already defined logger objects, define them properly where needed. - Convert one more unicode object into (Python3) string. - x2gobroker/tests/test_broker_agent.py: Assure that tests are run without loadchecker usage. - broker-use-load-checker profile option: Also tolerate 'TRUE' and 'True'. - x2gobroker/agent.py: Fix failing execution of LOCAL broker agent. As the LOCAL broker agent is executed setuid root, we cannot Popen.terminate() (which is unneeded anyway) the process after its execution. - Ignore SSH broker events for now. Not sure if we will ever support that. - Finalize API documentation. - Fix regression flaw in x2gobroker/web/json.py, introduced by commit 9fa371e9. * debian/*: + Trigger Makefile's install target and install those files. Drop debhelper from-source-installation magic. * debian/{control,compat}: Bump to DH version level 9. * debian/{control,x2gobroker-common.install}: + Split out common files into non-Pythonian bin:pkg. * debian/*.install: + Add EOLs at EOF. + Add tmpfiles.d files into bin:pkgs. + Fix installation to /usr/lib/python3.x paths. * debian/control: + Drop from D (several bin:pkgs): python3-argparse, argparse is shipped with Python3 core. + Switch from libapache2-mod-wsgi to libapache2-mod-wsgi-py3. + Add B-D: dh-python. + Add B-D: python3-netaddr (for unit tests). * debian/x2gobroker-loadchecker.postinst: + Do chown/chmod on the correct file (not authservice.log, but loadchecker.log). * debian/python-x2gobroker-doc.doc-base: + Drop leading white-space in Abstract: field. * x2gobroker.spec: + Adapt to Python3 port. + Bump package version. + CentOS 6 + 7 have python34-devel, not python3-devel. + Enable debug_packages for openSUSE Tumbleweed (suse_version > 1500). + CentOS 6 + 7 have python34-setuptools, not python3-setuptools. + Fix removal of conf files in tmpfiles.d where needed. + Install tmpfiles.d configs into bin:pkgs. + Only install tmpfiles.d configs on systems that support/have systemd. + Some path fixes for the new tmpfiles.d/. + Make sure the build chroot has all it needs to run the PyModule's unit tests. + Let's try to get unit tests working on Fedora first... . [ Mihai Moldovan ] * New upstream version (0.0.4.0): - src/x2gobroker-{agent,ssh}.c: catch errors in setuid wrappers and add general return clause to make compilers happy. - Makefile: make sure that we actually append our custom CFLAGS and LDFLAGS values, even if passed in through the make command line. - src/x2gobroker-{agent,ssh}.c: fix compile warnings/errors. - src/x2gobroker-{agent,ssh}.c: fix more compile errors. - misc: copyright update. - misc: switch to HTTPS-based URLs where appropriate. - man/*: update date and version stamps pre-release. - misc: add missing coding modelines. * x2gobroker.spec: - Add %debug_package macro when debugging is to be enabled, hoping that it will actually generate proper debuginfo (and -source) sub packages owning files. - Whitespace only. - Remove obsolete EPEL 5 support. - Switch to HTTPS-based links. - Use more curly braces. - Pull in gcc and redhat-rpm-config. - Re-enable debug file generation to see which OS versions still fail. - %exclude does not work with curly braces, revert. - Remove %debug_package macro usage, breaks builds nowadays. - Pass down global flags in CFLAGS and LDFLAGS. - Fix %{__global_ldflags} usage if variable does not exist. - Commands don't seem to work when wrapped in curly braces (at least on *SuSE), so revert. From unknown Thu Mar 28 12:23:34 2024 MIME-Version: 1.0 X-Mailer: MIME-tools 5.507 (Entity 5.507) X-Loop: owner@bugs.x2go.org From: owner@bugs.x2go.org (X2Go Bug Tracking System) Subject: Bug#1013 closed by X2Go Release Manager X2Go Release Manager (X2Go issue (in src:x2gobroker) has been marked as closed) Message-ID: References: <20190202210825.4CF6B5DAED@ymir.das-netzwerkteam.de> X-X2go-PR-Keywords: pending X-X2go-PR-Message: they-closed 1013 X-X2go-PR-Package: x2gobroker-daemon X-X2go-PR-Source: x2gobroker Date: Sat, 02 Feb 2019 21:10:16 +0000 Content-Type: multipart/mixed; boundary="----------=_1549141816-32059-0" This is a multi-part message in MIME format... ------------=_1549141816-32059-0 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 This is an automatic notification regarding your Bug report which was filed against the x2gobroker-daemon package: #1013: x2gobroker-daemon doesn't return error when it can't spawn/bind to p= ort It has been closed by X2Go Release Manager X2Go Release Manager . Their explanation is attached below along with your original report. If this explanation is unsatisfactory and you have not received a better one in a separate message then please contact X2Go Release Manager X= 2Go Release Manager by replying to this email. --=20 X2Go Bug Tracking System Contact owner@bugs.x2go.org with problems ------------=_1549141816-32059-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at control) by bugs.x2go.org; 2 Feb 2019 21:08:51 +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 4CF6B5DAED; Sat, 2 Feb 2019 22:08:25 +0100 (CET) From: =?utf-8?q?X2Go_Release_Manager?= X2Go Release Manager To: 1013-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 1013@bugs.x2go.org Subject: X2Go issue (in src:x2gobroker) has been marked as closed Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Message-Id: <20190202210825.4CF6B5DAED@ymir.das-netzwerkteam.de> Date: Sat, 2 Feb 2019 22:08:25 +0100 (CET) close #1013 thanks Hello, we are very hopeful that X2Go issue #1013 reported by you has been resolved in the new release (0.0.4.0) of the X2Go source project »src:x2gobroker«. You can view the complete changelog entry of src:x2gobroker (0.0.4.0) below, and you can use the following link to view all the code changes between this and the last release of src:x2gobroker. http://code.x2go.org/gitweb?p=x2gobroker.git;a=commitdiff;h=a2455880e34e31546054ce50abd1512c61430b51;hp=dbea0c7c20c58e6783ea796691f0881131ad6590 If you feel that the issue has not been resolved satisfyingly, feel free to reopen this bug report or submit a follow-up report with further observations described based on the new released version of src:x2gobroker. Thanks a lot for contributing to X2Go!!! light+love X2Go Git Admin (on behalf of the sender of this mail) --- X2Go Component: src:x2gobroker Version: 0.0.4.0-0x2go1 Status: RELEASE Date: Sat, 02 Feb 2019 21:50:29 +0100 Fixes: 1013 1234 1240 1252 1315 Changes: x2gobroker (0.0.4.0-0x2go1) RELEASED; urgency=medium . [ Mike Gabriel ] * New upstream version (0.0.4.0): - Bump upstream version to 0.0.4.0. - Port to Python 3. (Fixes: #1240). - Drop left-over debug print() call. - Makefile: Assure that setup.py is run under Python3. - Improve debugging messages during authentication phase. - x2gobroker/basicauth.py: Fix call of base64.decodestring on Python3. - Unit tests: Fix deep misunderstanding in the way allow-deny vs. deny-allow should actually work. - x2gobroker/brokers/base_broker.py: Entire rewrite of check_profile_acls() method. (Fixes: #1234). - x2gobroker/tests/test_web_plain_base.py: Add test case for passwords with accentuated characters (using the testsuite_authmech for now). - Makefile: Support skipping installation of the x2gobroker PyModule. Useful when building with CDBS on Debian. - Makefile: Compress man pages. - Makefile: Run setup.py build at build time. - tmpfiles.d utilization: Create RUNDIR/x2gobroker via tmpfiles.d system. Fixes missing dir and flawed permissions when running under systemd. - etc/x2gobroker.conf: Mention the per-profile option for enabling/disabling load checker support. - sbin/{x2gobroker-pubkeyauthorizer,x2gobroker-keygen}: Use proper octal numbers for file permissions. - sbin/x2gobroker-pubkeyauthorizer: Fix key lookup in os.environ for Python3. - sbin/x2gobroker-pubkeyauthorizer: Some string/bytecode fixes for Python3. Plus urllib -> urllib.request. - sbin/x2gobroker-pubkeyauthorizer: Improve key integrity checker and move it further up. Plus one more Python2 -> Python3 issue fixed. - sbin/x2gobroker-pubkeyauthorizer: Drop unused binascii import. - x2gobroker-pubkeyauthorizer: Tiny Python2to3 fix. - load checker integration: Make the default-use-load-checker option work like all other default-* options. - uccs frontend: Convert datetime.datetime object to string before answering the http request with it. - x2gobroker/agent (check_load()): Bail out if no remote agent is given. - x2gobroker-testagent: Convert to Python3 (using 2to3 tool). - x2gobroker-loadchecker: Python3'ify iteration over dict keys. - x2gobroker/utils.py: Provide helper functions for pretty-formatting key fingerprints. - x2gobroker-keygen: Use new fingerprint formatting functions. - x2gobroker/agent.py: Bail out if no hostaddr contained in remote_agent. - x2gobroker/agent.py: No load-checking when remote_agent is set to 'LOCAL'. - x2gobroker/agent.py: Better sanity checks for remote_agent and its dict keys hostname and hostaddr. - x2gobroker/loadchecker.py: Report properly to the logger if we fail to obtain a load factor. - x2gobroker-loadchecker.service: loadchecker service needs to chuid to system user x2gobroker. (Fixes: #1252). - x2gobroker-loadchecker.service: File ownership should be x2gobroker:x2gobroker, too. - x2gobroker-loadchecker: No chown/chmod if we are not running as root (which is mostly the case). - x2gobroker/brokers/inifile_broker.py: Make sure profile['name'] has a fallback if not given in the session profile. - x2gobroker/brokers/inifile_broker.py: Also check for presence of 'host' and 'sshport'. - UCCS API change for X2Go Sessions: Rename "SessionType" to "Command". - obligatory profile keys: Move from inifile backend to UCCS frontend, as those requirements are frontend specific. - UCCS: Start working on API version 5. - x2gobroker/uccsjson.py: Hide private Python class properties from JSON dict (like ._api_version). - UCCS frontend: Fix API version check. - UCSS frontend: Propagate API version onwards to the X2GoServer JSON generator class. - infile broker backend: Fix handling of empty lists in session profile and session profile defaults. - etc/x2gobroker-wsgi.apache.*: Drop Apache2.2 support. - Log to system broker.log file when run via x2gobroker-ssh. - Getting started documentation: Rework document, convert to markdown, install into x2gobroker bin:pkg (on DEB based systems). - Makefile.docupload: Add apidoc target (running sphinx-apidoc). - docs/source: Initialize Sphinx API documentation's .rst files. - bin/x2gobroker: If binding the http server fails, a non-zero exit code should be returned. (Fixes: #1013). - x2gobroker/loadchecker.py: Don't re-read the x2gobroker.conf during each cycle of the load checking loop. Rather read it on service startup and require a service restart when x2gobroker.conf has been changed. - x2gobroker/loadchecker.py: Avoid rare cases where at the end of a load checking cycle a negative sleep time would have been calculated. (Fixes: #1315). Thanks to Walid Moghrabi for catching this. - HTTP broker: Add &login= support to plain and json broker frontends. - SSH broker: Add --login option. This now supports X2Go Broker user and X2Go Server username being different accounts. - bin/x2gobroker: Correctly use split_host_address() function call. - bin/x2gobroker: Don't override already defined logger objects, define them properly where needed. - Convert one more unicode object into (Python3) string. - x2gobroker/tests/test_broker_agent.py: Assure that tests are run without loadchecker usage. - broker-use-load-checker profile option: Also tolerate 'TRUE' and 'True'. - x2gobroker/agent.py: Fix failing execution of LOCAL broker agent. As the LOCAL broker agent is executed setuid root, we cannot Popen.terminate() (which is unneeded anyway) the process after its execution. - Ignore SSH broker events for now. Not sure if we will ever support that. - Finalize API documentation. - Fix regression flaw in x2gobroker/web/json.py, introduced by commit 9fa371e9. * debian/*: + Trigger Makefile's install target and install those files. Drop debhelper from-source-installation magic. * debian/{control,compat}: Bump to DH version level 9. * debian/{control,x2gobroker-common.install}: + Split out common files into non-Pythonian bin:pkg. * debian/*.install: + Add EOLs at EOF. + Add tmpfiles.d files into bin:pkgs. + Fix installation to /usr/lib/python3.x paths. * debian/control: + Drop from D (several bin:pkgs): python3-argparse, argparse is shipped with Python3 core. + Switch from libapache2-mod-wsgi to libapache2-mod-wsgi-py3. + Add B-D: dh-python. + Add B-D: python3-netaddr (for unit tests). * debian/x2gobroker-loadchecker.postinst: + Do chown/chmod on the correct file (not authservice.log, but loadchecker.log). * debian/python-x2gobroker-doc.doc-base: + Drop leading white-space in Abstract: field. * x2gobroker.spec: + Adapt to Python3 port. + Bump package version. + CentOS 6 + 7 have python34-devel, not python3-devel. + Enable debug_packages for openSUSE Tumbleweed (suse_version > 1500). + CentOS 6 + 7 have python34-setuptools, not python3-setuptools. + Fix removal of conf files in tmpfiles.d where needed. + Install tmpfiles.d configs into bin:pkgs. + Only install tmpfiles.d configs on systems that support/have systemd. + Some path fixes for the new tmpfiles.d/. + Make sure the build chroot has all it needs to run the PyModule's unit tests. + Let's try to get unit tests working on Fedora first... . [ Mihai Moldovan ] * New upstream version (0.0.4.0): - src/x2gobroker-{agent,ssh}.c: catch errors in setuid wrappers and add general return clause to make compilers happy. - Makefile: make sure that we actually append our custom CFLAGS and LDFLAGS values, even if passed in through the make command line. - src/x2gobroker-{agent,ssh}.c: fix compile warnings/errors. - src/x2gobroker-{agent,ssh}.c: fix more compile errors. - misc: copyright update. - misc: switch to HTTPS-based URLs where appropriate. - man/*: update date and version stamps pre-release. - misc: add missing coding modelines. * x2gobroker.spec: - Add %debug_package macro when debugging is to be enabled, hoping that it will actually generate proper debuginfo (and -source) sub packages owning files. - Whitespace only. - Remove obsolete EPEL 5 support. - Switch to HTTPS-based links. - Use more curly braces. - Pull in gcc and redhat-rpm-config. - Re-enable debug file generation to see which OS versions still fail. - %exclude does not work with curly braces, revert. - Remove %debug_package macro usage, breaks builds nowadays. - Pass down global flags in CFLAGS and LDFLAGS. - Fix %{__global_ldflags} usage if variable does not exist. - Commands don't seem to work when wrapped in curly braces (at least on *SuSE), so revert. ------------=_1549141816-32059-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at submit) by bugs.x2go.org; 22 Mar 2016 09:54:56 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=3.0 tests=BAYES_00 autolearn=ham version=3.3.2 Received: from localhost (localhost [127.0.0.1]) by ymir.das-netzwerkteam.de (Postfix) with ESMTP id 1E6825DAB9 for ; Tue, 22 Mar 2016 10:54:55 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at ymir.das-netzwerkteam.de Received: from ymir.das-netzwerkteam.de ([127.0.0.1]) by localhost (ymir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 8gkpw6TGMqGM for ; Tue, 22 Mar 2016 10:54:49 +0100 (CET) Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.126.134]) by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id F226D5DA92 for ; Tue, 22 Mar 2016 10:54:48 +0100 (CET) Received: from [192.168.0.171] ([109.193.81.164]) by mrelayeu.kundenserver.de (mreue004) with ESMTPSA (Nemesis) id 0MbtgI-1aSQ2A2Wms-00JKne for ; Tue, 22 Mar 2016 10:54:48 +0100 To: submit@bugs.x2go.org From: Stefan Baur Subject: x2gobroker-daemon doesn't return error when it can't spawn/bind to port Message-ID: <56F11663.5060902@baur-itcs.de> Date: Tue, 22 Mar 2016 10:54:43 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.7.0 MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="UHwSxnslJLA1p401MB9OJLQphx9EKwqI6" X-Provags-ID: V03:K0:CeQID6aH+z2XoTfiFMmwDQiVNhXNR4PMrMuEGomYySrh8BZU+Xw pfIT+qC00CE6AY218fCU6VNGIT+zC4RhlLoyrRYHfpmbe4E09z3AHgj/3NMFgkcT0OLxsuJ N6aefaGtJ2ddmmF5HmqAJF+8XD7mkF5ihyrb/gE2jEhlNpg6jEscWMC/MV3GWdEDg4HqJ1A K0+8PtH0xrKqp2eB8OyOA== X-UI-Out-Filterresults: notjunk:1;V01:K0:C7uEFLZO5Wc=:eDnnU4Qjh8fVkpIQRy8ItO KW2qflIzntpSds67R4FCKFmsdbCUM3r4bsDrdLpUn6j5jnhGnfxc/f0sW2Mzm3i2N9ndEK5xG 3TPWbAhk7H+4LMtXiHwzFvNDq3z8JbPMLE/bBNzYfANoFFwjzJ0YBOvfyg4/jrDWuVoIAqsJo XHoivS4k8Z4Q6znFPLQnmzoKObY4uSK5pe2Xqqs3QnMCBGgc70gRsVx3bGv6cwvu4qpp4K3C8 JYlGRWD2HLUa9ligulrX2U0UJmw2QH24gNWdlPaWmA76uxIUKiKIsoCda+AzFY8RhIPwizxD/ 6HSJ1XZiKXbh6hNZ83lBui4Vurdbgnajo3BCmApNugnfmJ5BWvqjsY6vHsWTWwDmrPFyKbLbq VT43QM7IN9W3mNt2qMuphbA+jfieK079JJLdTSdmG+w/6pFmfjTMetVk1qrPq78DsDAiMTYsP PswBMUhOwgAUXK8TP4xD8LWgHEpyZaZ1HtThTJ+5WoUwkFTnja+3eqXU/zJukp9VKpTJv0zT+ oa1sXFOtILTjoZtDQFS9azofX9Oi8b+mr+tBPq+IP7ue24OYa03tyG6mFoVsg3dViOU5glpgr SnrQy2XpqBywlRW25ABfKgrbW2VSVQ9nQ/7yxfd0Xtk+zcVlgINO8PElS4PPPQjYEeD0FjHwz 8Sbg4xzrY+XOpyg5raCWC9/d6bmAVliWfTe5wlS3rb8iBCI2UanZDyhume+FadvJOefQ= This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --UHwSxnslJLA1p401MB9OJLQphx9EKwqI6 Content-Type: multipart/mixed; boundary="OoHjjxj9T4Bp3wDBh959PMn0P1UxXXd6g" From: Stefan Baur To: submit@bugs.x2go.org Message-ID: <56F11663.5060902@baur-itcs.de> Subject: x2gobroker-daemon doesn't return error when it can't spawn/bind to port --OoHjjxj9T4Bp3wDBh959PMn0P1UxXXd6g Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable package: x2gobroker-daemon version: 0.0.3.0 This is what happened: node:~# service x2gobroker-daemon stop [....] Stopping X2Go Session Broker standalone daemon: x2gobroker-daemonstart-stop-daemon: warning: failed to kill 14170: No such process failed! # huh, wait, what? node:~# ps ax | grep brok 5499 ? S 0:00 x2gobroker-authservice -s /run/x2gobroker/x2gobroker-authservice.socket -o root -g x2gobroker -p 06= 60 18791 pts/0 S+ 0:00 grep brok # no running x2gobroker-daemon, so ... # let's remove the PID file: node:~# rm /run/x2gobroker/x2gobroker-daemon.pid node:~# service x2gobroker-daemon stop node:~# service x2gobroker-daemon start [ ok ] Starting X2Go Session Broker standalone daemon: x2gobroker-daemon.= # looking good so far ... node:~# service x2gobroker-daemon stop [....] Stopping X2Go Session Broker standalone daemon: x2gobroker-daemonstart-stop-daemon: warning: failed to kill 18470: No such process failed! # whoops, again? node:~# ps ax | grep brok 5499 ? S 0:00 x2gobroker-authservice -s /run/x2gobroker/x2gobroker-authservice.socket -o root -g x2gobroker -p 06= 60 18791 pts/0 S+ 0:00 grep brok # no running x2gobroker-daemon, so ... # let's remove the PID file: node:~# rm /run/x2gobroker/x2gobroker-daemon.pid # I wonder what a restart will do? node:~# service x2gobroker-daemon restart [ ok ] Starting X2Go Session Broker standalone daemon: x2gobroker-daemon.= node:~# service x2gobroker-daemon restart [....] Stopping X2Go Session Broker standalone daemon: x2gobroker-daemonstart-stop-daemon: warning: failed to kill 18641: No such process failed! [warn] X2Go Session Broker: stale PID file (/run/x2gobroker/x2gobroker-daemon.pid). Delete it manually! ... (warning= ). node:~# ps ax | grep brok 5499 ? S 0:00 x2gobroker-authservice -s /run/x2gobroker/x2gobroker-authservice.socket -o root -g x2gobroker -p 06= 60 18791 pts/0 S+ 0:00 grep brok # and again, no running x2gobroker-daemon :-( The reason, as I found out later, was, that it tries to bind to 127.0.0.1:8080 by default, and that port is already in use by another process on that machine. IMO, this is a bug: when it can't bind to the port, and thus dies, this should be returned to the start script, so it can report back "FAILED". -Stefan --=20 BAUR-ITCS UG (haftungsbeschr=C3=A4nkt) Gesch=C3=A4ftsf=C3=BChrer: Stefan Baur Eichen=C3=A4ckerweg 10, 89081 Ulm | Registergericht Ulm, HRB 724364 Fon/Fax 0731 40 34 66-36/-35 | USt-IdNr.: DE268653243 --OoHjjxj9T4Bp3wDBh959PMn0P1UxXXd6g-- --UHwSxnslJLA1p401MB9OJLQphx9EKwqI6 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJW8RZnAAoJEG7d9BjNvlEZS00IAKvdn7OfKs2xzQ7mU14c48h3 KJbdvtFpnTcZ6MVjQ+J7YLmJHgOE0S9q36xz8gVMGz42r1gEO1SyWEX1AMhEbjNY DCNzOk1YrKlVsXdH8zLvVidzSazZIN5TsPhXUSP03p/Av1DFnkOoDuKnFrNTTMa3 uHBj7vvsYVMvvie1E+d/43/gX9Vglltncig0G/VE7brj91/kqurF9VpBetQWg3iU F4rqU3tpI+g+DZeWJPoMBhiNApHIu2DQn2f4ntLbbd/9PtT//kUOkuBt43qJQ2VT vjuW/Oj7FVwyX9mTNvCWh4qd7yCGyrSFTWL8G7tKGNek3WHgmUHaiUZHoTVtljA= =W9jw -----END PGP SIGNATURE----- --UHwSxnslJLA1p401MB9OJLQphx9EKwqI6-- ------------=_1549141816-32059-0-- From unknown Thu Mar 28 12:23:34 2024 Received: (at fakecontrol) by fakecontrolmessage; To: internal_control@bugs.x2go.org From: Debbugs Internal Request Subject: Internal Control Message-Id: Bug archived. Date: Sun, 03 Mar 2019 06:24:01 +0000 User-Agent: Fakemail v42.6.9 # A New Hope # A long time ago, in a galaxy far, far away # something happened. # # Magically this resulted in the following # action being taken, but this fake control # message doesn't tell you why it happened # # The action: # Bug archived. thanks # This fakemail brought to you by your local debbugs # administrator