#1469 - X2Go Client connection issue (Socket error)

X2Go Bug report logs - #1469
X2Go Client connection issue (Socket error)

Reported by: Melroy van den Berg <melroy89@protonmail.com>

Date: Mon, 18 May 2020 18:55:02 UTC

Severity: normal

Tags: pending

Found in version 4.1.2.2

Fixed in version 4.1.2.3

Done: X2Go Release Manager X2Go Release Manager <git-admin@x2go.org>

Bug is archived. No further changes may be made.

X-Loop: owner@bugs.x2go.org
Subject: Bug#1469: Re: [X2Go-Dev] Bug#1469: But wait; there is more... patches!
Reply-To: Melroy van den Berg <melroy89@protonmail.com>, 1469@bugs.x2go.org
Resent-From: Melroy van den Berg <melroy89@protonmail.com>
Resent-To: x2go-dev@lists.x2go.org
Resent-CC: X2Go Developers <x2go-dev@lists.x2go.org>
X-Loop: owner@bugs.x2go.org
Resent-Date: Wed, 20 May 2020 15:10:15 +0000
Resent-Message-ID: <handler.1469.B1469.158998719616788@bugs.x2go.org>
Resent-Sender: owner@bugs.x2go.org
X-X2Go-PR-Message: followup 1469
X-X2Go-PR-Package: x2goclient
X-X2Go-PR-Keywords: pending
References: <vq79NxFEEt8Pcp8OgUNEMq3mjc-pTXr2Am-M5IFBFEQ3Z2QlGHllLVmhjXXGe6fRM_hIfq91T7Y419CPz62rBb9Yn0GQL6Fc2s1kdrffjlY=@protonmail.com>
Received: via spool by 1469-submit@bugs.x2go.org id=B1469.158998719616788
          (code B ref 1469); Wed, 20 May 2020 15:10:15 +0000
Received: (at 1469) by bugs.x2go.org; 20 May 2020 15:06:36 +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.0 required=3.0 tests=BAYES_50,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,FREEMAIL_REPLYTO_END_DIGIT,
	HTML_MESSAGE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,URIBL_BLOCKED
	autolearn=no autolearn_force=no version=3.4.2
Received: from mail-40140.protonmail.ch (mail-40140.protonmail.ch [185.70.40.140])
	by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id 626755DAF8
	for <1469@bugs.x2go.org>; Wed, 20 May 2020 17:04:26 +0200 (CEST)
Date: Wed, 20 May 2020 15:04:17 +0000
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonmail.com;
	s=protonmail; t=1589987065;
	bh=AmLdDkqrcW1uvS9HobWfYoy78fRrR5nqAHt6Uz2nzZQ=;
	h=Date:To:From:Reply-To:Subject:From;
	b=DelDzePipKUBVoQ6OgOp+MgT9XVN6indUUTb9BIgJlICdtQxrkX56c0NinJEbPp35
	 /dMbiT11Xv/YJUAl4ZCbCExTSozkWhM5RAO8J+BxY9NYUtyX5Z2LkCf0xgB3HpBpdj
	 1NjQqX7dIU9WebXIljQU3zS9nVoTozDv7k1O8VkM=
To: "1469@bugs.x2go.org" <1469@bugs.x2go.org>
From: Melroy van den Berg <melroy89@protonmail.com>
Message-ID: <aXzJi4lDef700vOhX28Gr5WC1lkeYFfZHXswy-JpFf0RXhTBanWTn-cU43ol4szfw8U_d_u2utTEewcEarBUGbOUN_BXSlZtsWyO8mcSjc8=@protonmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="b1_11cf3dce7a98bf001db7f11ea287db30"

[Message part 1 (text/plain, inline)]

Hi Mike, Thanks for your reply. Are are my thoughts. > Using (std::)endl is actually a bug Ah, thanks for the heads-up! I think in that case we should in fact remove all the endl instead of adding it :). > Strictly speaking, though, this should not cause any issues Why was I getting socket error then? I really hope everything got merged correctly now. Including my changes in the separate function. Since all the changes together made my socket error from libssh disappear now. > This code path is for old libssh versions only (pre-0.8) and still works there, > so I don't see any reason to change it. I got deprecation warnings on this function/line while compiling. Why not merge it and remove build warnings? The change in within the #else, maybe if you think it should be in the #if you can apply the change in that section (but I think in my case the else part got compiled). Nevertheless, it's a deprecation that needs to be solved anyway. > Not changing return (...) vs. return ..., the actual change > looks good though. Why not changing the return(). This is wrong code, it's just a boolean! Bad practice, please don't it. Don't be afraid to change it. A generic remark: I think X2Go is missing a good pipeline with testcases and other quality checks. Which also hopefully increases your *trust in the code* and enables refactoring as well. I'm not afraid to refactor the code and clean-up the formatting, splitting functions and even into multiple files. If this all improve readability, debuggability and test-ability long-term. That is why I raise a request to create a decent pipeline to allow the necessary changes in further improve code maturity and the needed changes to do so. And maybe even a better diff tool to perform refactoring changes during review. Any ideas or suggestions? I'm running a GitLab instance myself for example; which enables DevOps and CI/CD within all my projects. Thanks once more! Kind regards, Melroy van den Berg

[Message part 2 (text/html, inline)]

X2Go Developers <owner@bugs.x2go.org>. Last modified: Thu Apr 18 10:16:52 2024; 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/.

X2Go Bug report logs - #1469 X2Go Client connection issue (Socket error)

X2Go Bug report logs - #1469
X2Go Client connection issue (Socket error)