From 8270ef0b8861bfc6d7a4e5bbe4e6125a221d0680 Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Mon, 22 Jul 2013 08:50:03 -0400
Subject: [PATCH 0/6] Storing authentication info in session.
The authenticator configuration has been modified to store the authentication
info in the session so it can be used by the servlets. An upgrade script has
been added to update the configuration in existing instances.
The SSLAuthenticatorWithFalback was modified to propagate the configuration
to the actual authenticator handling the request.
---
base/ca/shared/webapps/ca/META-INF/context.xml | 4 +-
.../cms/tomcat/SSLAuthenticatorWithFallback.java | 5 ++
base/kra/shared/webapps/kra/META-INF/context.xml | 4 +-
base/ocsp/shared/webapps/ocsp/META-INF/context.xml | 4 +-
base/server/upgrade/10.0.4/.gitignore | 4 ++
.../upgrade/10.0.5/01-EnableSessionInAuthenticator | 69 ++++++++++++++++++++++
base/tks/shared/webapps/tks/META-INF/context.xml | 4 +-
7 files changed, 90 insertions(+), 4 deletions(-)
create mode 100644 base/server/upgrade/10.0.4/.gitignore
create mode 100755 base/server/upgrade/10.0.5/01-EnableSessionInAuthenticator
diff --git a/base/ca/shared/webapps/ca/META-INF/context.xml b/base/ca/shared/webapps/ca/META-INF/context.xml
index 032fd14..e838503 100644
--- a/base/ca/shared/webapps/ca/META-INF/context.xml
+++ b/base/ca/shared/webapps/ca/META-INF/context.xml
@@ -28,7 +28,9 @@
secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
<Valve className="com.netscape.cms.tomcat.SSLAuthenticatorWithFallback"
- secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
+ alwaysUseSession="true"
+ secureRandomProvider="Mozilla-JSS"
+ secureRandomAlgorithm="pkcs11prng"/>
<Realm className="com.netscape.cms.tomcat.ProxyRealm" />
diff --git a/base/common/src/com/netscape/cms/tomcat/SSLAuthenticatorWithFallback.java b/base/common/src/com/netscape/cms/tomcat/SSLAuthenticatorWithFallback.java
index d1b3dc3..20bf85d 100644
--- a/base/common/src/com/netscape/cms/tomcat/SSLAuthenticatorWithFallback.java
+++ b/base/common/src/com/netscape/cms/tomcat/SSLAuthenticatorWithFallback.java
@@ -140,8 +140,13 @@ public class SSLAuthenticatorWithFallback extends AuthenticatorBase {
@Override
protected void initInternal() throws LifecycleException {
log("Initializing authenticators");
+
super.initInternal();
+
+ sslAuthenticator.setAlwaysUseSession(alwaysUseSession);
sslAuthenticator.init();
+
+ fallbackAuthenticator.setAlwaysUseSession(alwaysUseSession);
fallbackAuthenticator.init();
}
diff --git a/base/kra/shared/webapps/kra/META-INF/context.xml b/base/kra/shared/webapps/kra/META-INF/context.xml
index 032fd14..e838503 100644
--- a/base/kra/shared/webapps/kra/META-INF/context.xml
+++ b/base/kra/shared/webapps/kra/META-INF/context.xml
@@ -28,7 +28,9 @@
secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
<Valve className="com.netscape.cms.tomcat.SSLAuthenticatorWithFallback"
- secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
+ alwaysUseSession="true"
+ secureRandomProvider="Mozilla-JSS"
+ secureRandomAlgorithm="pkcs11prng"/>
<Realm className="com.netscape.cms.tomcat.ProxyRealm" />
diff --git a/base/ocsp/shared/webapps/ocsp/META-INF/context.xml b/base/ocsp/shared/webapps/ocsp/META-INF/context.xml
index 032fd14..e838503 100644
--- a/base/ocsp/shared/webapps/ocsp/META-INF/context.xml
+++ b/base/ocsp/shared/webapps/ocsp/META-INF/context.xml
@@ -28,7 +28,9 @@
secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
<Valve className="com.netscape.cms.tomcat.SSLAuthenticatorWithFallback"
- secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
+ alwaysUseSession="true"
+ secureRandomProvider="Mozilla-JSS"
+ secureRandomAlgorithm="pkcs11prng"/>
<Realm className="com.netscape.cms.tomcat.ProxyRealm" />
diff --git a/base/server/upgrade/10.0.4/.gitignore b/base/server/upgrade/10.0.4/.gitignore
new file mode 100644
index 0000000..5e7d273
--- /dev/null
+++ b/base/server/upgrade/10.0.4/.gitignore
@@ -0,0 +1,4 @@
+# Ignore everything in this directory
+*
+# Except this file
+!.gitignore
diff --git a/base/server/upgrade/10.0.5/01-EnableSessionInAuthenticator b/base/server/upgrade/10.0.5/01-EnableSessionInAuthenticator
new file mode 100755
index 0000000..7aee780
--- /dev/null
+++ b/base/server/upgrade/10.0.5/01-EnableSessionInAuthenticator
@@ -0,0 +1,69 @@
+#!/usr/bin/python
+# Authors:
+# Endi S. Dewata <edewata@redhat.com>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; version 2 of the License.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+#
+# Copyright (C) 2013 Red Hat, Inc.
+# All rights reserved.
+#
+
+import os
+from lxml import etree
+
+import pki.server.upgrade
+
+
+class EnableSessionInAuthenticator(pki.server.upgrade.PKIServerUpgradeScriptlet):
+
+ def __init__(self):
+
+ self.message = 'Enable session in authenticator'
+
+ self.parser = etree.XMLParser(remove_blank_text=True)
+
+ def upgrade_subsystem(self, instance, subsystem):
+
+ context_xml = os.path.join(
+ instance.base_dir, 'webapps', subsystem.name, 'META-INF', 'context.xml')
+ self.backup(context_xml)
+
+ document = etree.parse(context_xml, self.parser)
+
+ self.enable_session(document)
+
+ with open(context_xml, 'w') as f:
+ f.write(etree.tostring(document, pretty_print=True))
+
+ def enable_session(self, document):
+
+ context = document.getroot()
+ valves = context.findall('Valve')
+ authenticator = None
+
+ # Find existing authenticator
+ for valve in valves:
+ className = valve.get('className')
+ if className != 'com.netscape.cms.tomcat.SSLAuthenticatorWithFallback':
+ continue
+
+ # Found existing authenticator
+ authenticator = valve
+ break
+
+ if authenticator is None:
+ raise Exception('Missing SSLAuthenticatorWithFallback')
+
+ # Update authenticator's attributes
+ authenticator.set('alwaysUseSession', 'true')
diff --git a/base/tks/shared/webapps/tks/META-INF/context.xml b/base/tks/shared/webapps/tks/META-INF/context.xml
index 032fd14..e838503 100644
--- a/base/tks/shared/webapps/tks/META-INF/context.xml
+++ b/base/tks/shared/webapps/tks/META-INF/context.xml
@@ -28,7 +28,9 @@
secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
<Valve className="com.netscape.cms.tomcat.SSLAuthenticatorWithFallback"
- secureRandomProvider="Mozilla-JSS" secureRandomAlgorithm="pkcs11prng"/>
+ alwaysUseSession="true"
+ secureRandomProvider="Mozilla-JSS"
+ secureRandomAlgorithm="pkcs11prng"/>
<Realm className="com.netscape.cms.tomcat.ProxyRealm" />
--
1.8.3.1