[X2Go-Commits] [x2gobroker] 01/01: Revert "Allow switching to dynamic cookie auth, after the connection has been successfully established by a first successful user+password authentication."
git-admin at x2go.org
git-admin at x2go.org
Sun Mar 9 21:13:16 CET 2014
This is an automated email from the git hooks/post-receive script.
x2go pushed a commit to branch master
in repository x2gobroker.
commit f1382f48ffb408c7904874e15978c3f4c59538db
Author: Mike Gabriel <mike.gabriel at das-netzwerkteam.de>
Date: Sun Mar 9 21:12:42 2014 +0100
Revert "Allow switching to dynamic cookie auth, after the connection has been successfully established by a first successful user+password authentication."
This reverts commit 31e2b0c99d26a5061fbb4c85fd86f0d229decfc5.
Conflicts:
debian/changelog
x2gobroker/brokers/base_broker.py
---
debian/changelog | 3 ---
etc/x2gobroker.conf | 9 ---------
x2gobroker/brokers/base_broker.py | 6 ++++--
x2gobroker/defaults.py | 1 -
4 files changed, 4 insertions(+), 15 deletions(-)
diff --git a/debian/changelog b/debian/changelog
index ac7f2aa..1a88420 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -93,9 +93,6 @@ x2gobroker (0.0.3.0-0x2go1) UNRELEASED; urgency=low
x2gobroker(-daemon) is run as uidNumber 0.
- Implement dynamic authid for JSON WebUI frontend. Add a generic metadata
top level to the JSON output tree.
- - Allow switching to dynamic cookie auth, after the connection has been
- successfully established by a first successful user+password
- authentication.
- Store cookies in /var/lib/x2gobroker (path is more appropriate than
previously suggested path /var/log/x2gobroker).
* debian/control:
diff --git a/etc/x2gobroker.conf b/etc/x2gobroker.conf
index 33f8ccf..730ca03 100644
--- a/etc/x2gobroker.conf
+++ b/etc/x2gobroker.conf
@@ -34,15 +34,6 @@
# to make the X2Go Session Broker require this feature
#require-cookie = false
-# If require-cookie is set to true, the user normally always needs an initial
-# authentication cookie to start the connection with (this is pre-shared key
-# the broker admin has to provide to X2Go Client users).
-#
-# Alternatively, you can allow a first authentication without cookie, but with
-# username and password only. Further authentication then can use the cookie
-# provided after first successful connect.
-#password-auth-initializes-cookie=true
-
# X2Go supports two different cookie authentication modes (static and dynamic).
# Dynamic cookies send new cookie to client on every request. This could possibly
# cause issues if a client ever tries multiple requests at the same time.
diff --git a/x2gobroker/brokers/base_broker.py b/x2gobroker/brokers/base_broker.py
index 66e92e5..f8445a3 100644
--- a/x2gobroker/brokers/base_broker.py
+++ b/x2gobroker/brokers/base_broker.py
@@ -755,6 +755,8 @@ class X2GoBroker(object):
# this catches a validation check from the UCCS web frontend...
return False, None
+ ### IMPLEMENT YOUR AUTHENTICATION LOGIC IN THE self._do_authenticate(**kwargs) METHOD
+ ### when inheriting from the base.X2GoBroker class.
if type(cookie) is types.StringType:
cookie = unicode(cookie)
@@ -781,8 +783,8 @@ class X2GoBroker(object):
logger_broker.warning('base_broker.X2GoBroker.check_access(): could not create cookie-directory {cookie_directory} failing to authenticate'.format(cookie_directory=cookie_directory))
return False, None
- if access or cookie == None or cookie == "" or self.config.get_value('global', 'password-auth-initializes-cookie'):
- # this should be the first time we have seen this user or they are using old client so verify their password
+ if access or cookie == None or cookie == "":
+ # this should be the first time we have seen this user or they are using old client so verify their passwrd
### IMPLEMENT YOUR AUTHENTICATION LOGIC IN THE self._do_authenticate(**kwargs) METHOD
### when inheriting from the base.X2GoBroker class.
diff --git a/x2gobroker/defaults.py b/x2gobroker/defaults.py
index c1b3749..a652837 100644
--- a/x2gobroker/defaults.py
+++ b/x2gobroker/defaults.py
@@ -182,7 +182,6 @@ X2GOBROKER_CONFIG_DEFAULTS = {
'global': {
u'require-password': True,
u'require-cookie': False,
- u'password-auth-initializes-cookie': True,
u'use-static-cookie': True,
u'auth-timeout': 36000,
u'cookie-directory': '/var/lib/x2gobroker/cookies',
--
Alioth's /srv/git/_hooks_/post-receive-email on /srv/git/code.x2go.org/x2gobroker.git
More information about the x2go-commits
mailing list