X2Go Bug report logs - #876
Problem with numeric user id

version graph

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

Reported by: brian <brian@fodvo.org>

Date: Sat, 16 May 2015 19:10:02 UTC

Severity: normal

Found in version 4.01.19

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#876; Package x2goserver. (Sat, 16 May 2015 19:10:02 GMT) (full text, mbox, link).

Acknowledgement sent to brian <brian@fodvo.org>:
New Bug report received and forwarded. Copy sent to X2Go Developers <x2go-dev@lists.x2go.org>. (Sat, 16 May 2015 19:10:02 GMT) (full text, mbox, link).

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

From: brian <brian@fodvo.org>
To: submit@bugs.x2go.org
Subject: Problem with numeric user id
Date: Sat, 16 May 2015 14:05:11 -0500
[Message part 1 (text/plain, inline)]
Package: x2goserver
Version:  4.01.19

Server Fedora 21 x2goserver-
Client Fedora 20 x2goclient-

I had this working fine, and then one day the client would hang in a 
greyed out manner waiting to connect.  Several hours of tracing through 
the scripts, I learned the server x2gostartagent script was simply 
infinite looping through sessions and 
/usr/lib64/x2go/x2gosqlitewrapper.pl insertsession was rejecting them all.

I suspected the problem may be when I changed my user id on the server 
from alpha-numeric to all numeric...  Yes I know traditional UNIX/Linux 
user ids are supposed to start with at least 1 alpha character, but 
working for a large corp where MS Active Directory runs everything, they 
have successfully standardized all Windows and Linux user accounts to 6 
digit numerics.  You don't have to convince me this was a BAD idea, but 
way above my pay grade.  So when I changed my server to match (and yes 
Fedora will accept a numeric user id even without AD authentication, but 
it will warn against it), the failure started.  One more test I ran a 
connect as 'root' and it connected right away, so I knew where the 
problem was.

Digging through x2gosqlitewrapper.pl I noticed a lot of perl regex and 
found the sanitizer function, line 67.  When I replaced it with
 if ($string =~ /^([a-zA-Z0-9\_] ....  <adding the 0-9 to the match> 
everything started working again.

Diff file attached.
[x2gosqlitewrapper.pl.diff (text/x-patch, attachment)]

Send a report that this bug log contains spam.

X2Go Developers <owner@bugs.x2go.org>. Last modified: Sun May 26 02:25:11 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/.

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