X2Go Bug report logs - #514
Please add option to hide folder sharing options from x2goclient

Toggle useless messages

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

From: Stefan Baur <newsgroups.mail2@stefanbaur.de>

To: submit@bugs.x2go.org, Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

Subject: Please add option to hide folder sharing options from x2goclient

Date: Fri, 06 Jun 2014 19:58:19 +0200

Package: x2goclient
Severity: wishlist
Version: 4.0.2.0

Similar to Bugs http://bugs.x2go.org/cgi-bin/bugreport.cgi?bug=364 and
http://bugs.x2go.org/cgi-bin/bugreport.cgi?bug=365, I would like to
request an additional commandline parameter for x2goclient.

The parameter should be named --no-sharing or similar, and what it
should do is hide the folder sharing options

- in an active session, the "<Session Name>/Share Folder" entry on
  X2Go's taskbar icon context menu when minimized
- also in an active session, the "Share Folder" icon in the session
  window when not minimized
- in the session preferences windows, the "Shared folders" tab

The reason for this is that in environments where folder sharing is
disallowed by the admin (users not members of fuse group/fuse not
installed on server), it irritates the users, as those of the
"tinkering" type try to configure a shared folder, fail, and complain to
the admins. So it would be cool to be able to hide these options with a
command line switch.

Also, in environments where folder sharing *is* allowed, but where its
settings should not be changeable by the user, hiding the option and
making the config file readonly would be a way to keep users from making
unwanted changes.

-Stefan

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

From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

To: Stefan Baur <newsgroups.mail2@stefanbaur.de>

Cc: 514@bugs.x2go.org

Subject: Re: Please add option to hide folder sharing options from x2goclient

Date: Sat, 07 Jun 2014 13:10:09 +0000

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

Hi Stefan,

On  Fr 06 Jun 2014 19:58:19 CEST, Stefan Baur wrote:

> Similar to Bugs http://bugs.x2go.org/cgi-bin/bugreport.cgi?bug=364 and
> http://bugs.x2go.org/cgi-bin/bugreport.cgi?bug=365, I would like to
> request an additional commandline parameter for x2goclient.
>
> The parameter should be named --no-sharing or similar, and what it
> should do is hide the folder sharing options
>
> - in an active session, the "<Session Name>/Share Folder" entry on
>   X2Go's taskbar icon context menu when minimized
> - also in an active session, the "Share Folder" icon in the session
>   window when not minimized
> - in the session preferences windows, the "Shared folders" tab
>
> The reason for this is that in environments where folder sharing is
> disallowed by the admin (users not members of fuse group/fuse not
> installed on server), it irritates the users, as those of the
> "tinkering" type try to configure a shared folder, fail, and complain to
> the admins. So it would be cool to be able to hide these options with a
> command line switch.
>
> Also, in environments where folder sharing *is* allowed, but where its
> settings should not be changeable by the user, hiding the option and
> making the config file readonly would be a way to keep users from making
> unwanted changes.

Maybe this bug should be renamed to:

  "Test server-side group membership and hide sharing options if  
connecting user is not
   in group fuse"

???

Having such an option configurable client-side does only make a little  
sense too me, but not much.

Mike

-- 

DAS-NETZWERKTEAM
mike gabriel, 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/freebusy/m.gabriel%40das-netzwerkteam.de.xfb

[Message part 2 (application/pgp-signature, inline)]

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

From: Stefan Baur <newsgroups.mail2@stefanbaur.de>

To: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

Cc: 514@bugs.x2go.org

Subject: Re: Please add option to hide folder sharing options from x2goclient

Date: Sat, 07 Jun 2014 15:16:56 +0200

Am 07.06.2014 15:10, schrieb Mike Gabriel:

> Maybe this bug should be renamed to:
> 
> "Test server-side group membership and hide sharing options if 
> connecting user is not in group fuse"
> 
> ???
> 
> Having such an option configurable client-side does only make a
> little sense too me, but not much.

Logic flaw: You cannot check group membership before connecting.
Session preferences can be set in disconnected state, and that's where
the "Shared folders" tab is located.

So, you could split it up:

a) --no-sharing -> disables "Shared folders" tab in session preferences

b) After connect: not member of fuse group -> sharing-related
                                              icons/context menus
                                              disabled and hidden

-Stefan

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

From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

To: 514-submitter@bugs.x2go.org

Cc: control@bugs.x2go.org, 514@bugs.x2go.org

Subject: X2Go issue (in src:x2goclient) has been marked as pending for release

Date: Sat, 28 Jun 2014 23:33:07 +0200 (CEST)

tag #514 pending
fixed #514 4.0.2.1
thanks

Hello,

X2Go issue #514 (src:x2goclient) 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=x2goclient.git;a=commitdiff;h=0ebe95f

The issue will most likely be fixed in src:x2goclient (4.0.2.1).

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

---
commit 0ebe95fc2fa15a61ccbe4b66cb7b225fefd20e14
Author: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
Date:   Sat Jun 28 23:31:43 2014 +0200

    Add new cmdline option --hide-foldersharing. If used, all GUI elements related to exporting client-side folders to X2Go sessions get hidden. (Fixes: #514).

diff --git a/debian/changelog b/debian/changelog
index ff288f4..3fa1904 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -16,6 +16,9 @@ x2goclient (4.0.2.1-0x2go1) UNRELEASED; urgency=low
       For SSHd startup failures provide different messages on Windows
       and non-Windows machines. (Fixes: #235).
       FIXME: add detection code to report SSH daemon startup failures.
+    - Add new cmdline option --hide-foldersharing. If used, all GUI
+      elements related to exporting client-side folders to X2Go sessions
+      get hidden. (Fixes: #514).
   * debian/control:
     + Add dbg:package x2goplugin-dbg.
 

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

From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

To: 514-submitter@bugs.x2go.org

Cc: control@bugs.x2go.org, 514@bugs.x2go.org

Subject: X2Go issue (in src:x2goclient) has been marked as closed

Date: Sat, 12 Jul 2014 08:41:44 +0200 (CEST)

close #514
thanks

Hello,

we are very hopeful that X2Go issue #514 reported by you
has been resolved in the new release (4.0.2.1) of the
X2Go source project »src:x2goclient«.

You can view the complete changelog entry of src:x2goclient (4.0.2.1)
below, and you can use the following link to view all the code changes
between this and the last release of src:x2goclient.

    http://code.x2go.org/gitweb?p=x2goclient.git;a=commitdiff;h=6d9114eb5f414f34962f8b14f58e5d469acd1574;hp=85bf0c6e7539910fff779689528009a897cdceb4

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:x2goclient.

Thanks a lot for contributing to X2Go!!!

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

---
X2Go Component: src:x2goclient
Version: 4.0.2.1-0x2go1
Status: RELEASE
Date: Fri, 11 Jul 2014 12:23:59 +0200
Fixes: 235 364 365 421 478 498 502 514 528
Changes: 
 x2goclient (4.0.2.1-0x2go1) RELEASED; urgency=low
 .
   [ Mike Gabriel ]
   * New upstream release (4.0.2.1):
     - Fix FTBFS on arm64 (and others) architecture. (Fixes: #498).
     - Only show session name in notification bubbles with debugging
       enabled. (Fixes: #364).
     - New command line option (--keep-trayicon). With that options set,
       we force X2Go client to only use the default "accelerated X" as
       system tray icon (and prohibit usage of the session's icon as
       tray icon). (Fixes: #365).
     - Provide more meaningful messages on SSH errors (host pub key not
       found, export pub key not found, authorized_keys file not found).
       For SSHd startup failures provide different messages on Windows
       and non-Windows machines. (Fixes: #235).
       FIXME: add detection code to report SSH daemon startup failures.
     - Add new cmdline option --hide-foldersharing. If used, all GUI
       elements related to exporting client-side folders to X2Go sessions
       get hidden. (Fixes: #514).
     - Update COPYING file (to update FSF address).
   * debian/control:
     + Add dbg:package x2goplugin-dbg.
 .
   [ Clemens Lang ]
   * New upstream version (4.0.2.1):
     - Re-add Info.plist file (which got removed by accident in commit 91cf6c6).
       (Fixes: #502).
 .
   [ Oleksandr Shneyder ]
   * New upstream release (4.0.2.1):
     - Hide GSSAPI options on direct RDP connections. (Fixes: #478).
     - Search X-Server in /Applications/Utilities/XQuartz.app on Mac.
     - Wrong cmd line option throw error dialog if running not from terminal.
     - Add command line option "close-disconnect", which make client close after
       session is disconnected.
     - Don't start pulse on Windows, until we really have to start a session
       that requires pulse support.
     - Check if sound is activated before starting pulse.
     - Fix starting sshd on Win XP. (Fixes: #421).
     - Fix "fullscreen" mode on Windows 7 with multiple monitors.
 .
   [ Mike DePaulo ]
   * New upstream release (4.0.2.1):
     - Windows: Prevent high PulseAudio CPU usage on Windows XP by lowering
       PulseAudio's CPU priority from "high" to "normal" on XP specifically.
       Also do so on Windows Server 2003 (R2) (Fixes #526)
     - Windows: Enable PulseAudio log when --debug is passed.
     - Windows: Fix compatibility with current PulseAudio master branch
     - Windows: Reduce size of NSIS installer by switching to lzma solid compression
     - Windows: cygwin packages (inluding patched OpenSSH 6.6.1p1-3) updated from
       latest versions as of 2014-04-08 to latest versions as of 2014-06-09. This
       includes openssl 1.0.1h-1, which fixes the 6 CVEs announced on 2014-06-05.
       (Cygwin openssl was also individually updated to 1.0.1h-1 in
       4.0.2.0+build4.) This also includes migrating from libjpeg to
       libjpeg-turbo, which reduces CPU usage of nxproxy.)
     - Windows: Download and compile nx-libs-lite from the release tarball,
       rather than from git, so that the patches are actually applied
     - Windows: nx-libs-lite updated from 3.5.0.22 to 3.5.0.27
       (linked against aforementioned cygwin package versions)
     - Windows: Win32 OpenSSL updated from 1.0.1g to 1.0.1h, which fixes the 6
       CVEs announced on 2014-06-05.
       (Win32 OpenSSL was also updated to 1.0.1h in 4.0.2.0+build3.)
     - Windows: VcXsrv (XP Compatible) updated from 1.14.3.1 to 1.14.3.2.
       Fixes CVE-2014-0209, CVE-2014-0210, and CVE-2014-0211
       (VcXsrv was also updated to 1.14.3.2 in 4.0.2.0+build2.)
     - Windows: Switch from regular NSIS 2.46 to Unicode NSIS 2.46.5
       (Fixes: #528)

Send a report that this bug log contains spam.