[X2Go-Commits] [x2goserver] 01/04: Document session startup / resumption failures (and their reasons) in server-side log output.

[git-admin at x2go.org]

This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch release/4.0.1.x
in repository x2goserver.

commit f896085a9eb456da79b70a5619bbbf732e807fcf
Author: Mike Gabriel <mike.gabriel at das-netzwerkteam.de>
Date:   Mon Nov 17 10:24:00 2014 +0100

    Document session startup / resumption failures (and their reasons) in server-side log output.
---
 debian/changelog                  |    2 ++
 x2goserver/bin/x2goresume-session |   14 ++++++++++----
 x2goserver/bin/x2gostartagent     |   25 +++++++++++++++++++------
 3 files changed, 31 insertions(+), 10 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index e6edd03..6b59bad 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -47,6 +47,8 @@ x2goserver (4.0.1.19-0x2go1) UNRELEASED; urgency=medium
       up with that idea.
     - Fix cross-user X2Go Desktop Sharing after being broken by implementing
       clipboard mode feature (and probably other code changes).
+    - Document session startup / resumption failures (and their reasons) in
+      server-side log output.
   * debian/control:
     + Add D (x2goserver): libfile-which-perl.
   * debian/x2goserver.docs:
diff --git a/x2goserver/bin/x2goresume-session b/x2goserver/bin/x2goresume-session
index 8783dad..5f19176 100755
--- a/x2goserver/bin/x2goresume-session
+++ b/x2goserver/bin/x2goresume-session
@@ -20,10 +20,13 @@
 # Copyright (C) 2007-2014 Oleksandr Shneyder <oleksandr.shneyder at obviously-nice.de>
 # Copyright (C) 2007-2014 Heinz-Markus Graesing <heinz-m.graesing at obviously-nice.de>
 
+X2GO_LIB_PATH="$(x2gopath libexec)";
 
 if [ $# -lt 7 ]
 then
-	echo "not enough arguments"
+	msg="not enough command line arguments"
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -1
 fi
 SESSION_NAME="$1"
@@ -35,7 +38,6 @@ X2GO_KBD_TYPE="$6"
 X2GO_SET_KBD="$7"
 X2GO_CLIPBOARD="$8"
 
-X2GO_LIB_PATH="$(x2gopath libexec)";
 X2GO_AGENT_PID=`$X2GO_LIB_PATH/x2gogetagent "$SESSION_NAME"`
 
 X2GO_ROOT="${HOME}/.x2go"
@@ -56,7 +58,9 @@ STATE=`$X2GO_LIB_PATH/x2gogetagentstate "$SESSION_NAME"`
 # exit if session terminated
 if [ "$STATE" == "TERMINATED" ] || [ "$STATE" == "TERMINATING" ]
 then
-	echo "Session terminated"
+	msg="session $SESSION_NAME terminated"
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -1
 fi
 
@@ -92,7 +96,9 @@ do
 	i=$(($i+1))
 	if [ $i -gt 45 ]
 	then
-		echo "Error: taking too long to suspend session. It is possible that the session is damaged."
+		msg="it is taking too long to suspend the session-to-be-resumed; it is possible that the session is in a damaged state"
+		echo "$msg"
+		$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 		exit -1;
 	fi
 	STATE=`$X2GO_LIB_PATH/x2gogetagentstate "$SESSION_NAME"`
diff --git a/x2goserver/bin/x2gostartagent b/x2goserver/bin/x2gostartagent
index dde18cf..3964eb2 100755
--- a/x2goserver/bin/x2gostartagent
+++ b/x2goserver/bin/x2gostartagent
@@ -29,19 +29,29 @@ SSH_PORT=30000 #First ssh port 30001
 
 # some sanity checks before session startup...
 if egrep "^backend[ ]*=[ ]*postgres" /etc/x2go/x2gosql/sql 1>/dev/null 2>/dev/null && [ "x$USER" = "xroot" ]; then
-	echo "The super-user \"root\" is not allowed to launch X2Go sessions."
+	msg="The super-user \"root\" is not allowed to launch X2Go sessions."
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -1
 elif [ -z "$USER" ]; then
-	echo "The \$USER environment variable is not set. Aborting session startup."
+	msg="The \$USER environment variable is not set. Aborting session startup."
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -2
 elif [ -z "$SSH_CLIENT" ]; then
-	echo "The \$SSH_CLIENT environment variable is not set. Aborting session startup."
+	msg="The \$SSH_CLIENT environment variable is not set. Aborting session startup."
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -3
 elif [ -z "$HOME" ]; then
-	echo "The \$HOME environment variable is not set. Aborting session startup."
+	msg="The \$HOME environment variable is not set. Aborting session startup."
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -4
 elif ! echo $HOME | iconv -f ASCII -t ASCII 1>/dev/null 2>/dev/null; then
-	echo "Your home directory path contains non-ASCII characters. Aborting session startup."
+	msg="Your home directory path contains non-ASCII characters. Aborting session startup."
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -5
 fi
 
@@ -68,7 +78,9 @@ elif [ -z "$X2GO_CLIENT" ] && [ -n "$SSH_CONNECTION" ]; then
 	X2GO_CLIENT=`echo $SSH_CONNECTION | awk '{print $1}'`
 fi
 if [ -z "$X2GO_CLIENT" ]; then
-	echo "The \$X2GO_CLIENT environment variable is not set (due to neither \$SSH_CLIENT nor \$SSH_CONNECTION variable being set). Aborting session startup."
+	msg="The \$X2GO_CLIENT environment variable is not set (due to neither \$SSH_CLIENT nor \$SSH_CONNECTION variable being set). Aborting session startup."
+	echo "$msg"
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "$msg"
 	exit -3
 fi
 
@@ -149,6 +161,7 @@ LWORD=`echo $LIMIT | awk '{print $1}'`
 
 if [ "$LWORD" == "LIMIT" ]; then
 	echo  $LIMIT 1>&2
+	$X2GO_LIB_PATH/x2gosyslog "$0" "error" "session limit has been reached for user ,,$USER'', cannot start new session"
 	exit -1
 fi
 

--
Alioth's /srv/git/_hooks_/post-receive-email on /srv/git/code.x2go.org/x2goserver.git