Fix bug where passThruErrors setting was being ignored.
authorwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Fri, 12 Mar 2004 21:34:29 +0000 (21:34 +0000)
committerwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Fri, 12 Mar 2004 21:34:29 +0000 (21:34 +0000)
git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/trunk@914 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

src/edu/internet2/middleware/shibboleth/aa/AAConfig.java
src/edu/internet2/middleware/shibboleth/aa/AAServiceProviderMapper.java
src/edu/internet2/middleware/shibboleth/aa/AAServlet.java

index 5da8a98..233b365 100644 (file)
@@ -1,38 +1,27 @@
 /*
  * The Shibboleth License, Version 1. Copyright (c) 2002 University Corporation for Advanced Internet Development, Inc.
- * All rights reserved
- * 
- * 
- * Redistribution and use in source and binary forms, with or without modification, are permitted provided that the
- * following conditions are met:
- * 
- * Redistributions of source code must retain the above copyright notice, this list of conditions and the following
- * disclaimer.
- * 
- * Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following
- * disclaimer in the documentation and/or other materials provided with the distribution, if any, must include the
- * following acknowledgment: "This product includes software developed by the University Corporation for Advanced
- * Internet Development <http://www.ucaid.edu> Internet2 Project. Alternately, this acknowledegement may appear in the
- * software itself, if and wherever such third-party acknowledgments normally appear.
- * 
- * Neither the name of Shibboleth nor the names of its contributors, nor Internet2, nor the University Corporation for
- * Advanced Internet Development, Inc., nor UCAID may be used to endorse or promote products derived from this software
- * without specific prior written permission. For written permission, please contact shibboleth@shibboleth.org
- * 
- * Products derived from this software may not be called Shibboleth, Internet2, UCAID, or the University Corporation
- * for Advanced Internet Development, nor may Shibboleth appear in their name, without prior written permission of the
- * University Corporation for Advanced Internet Development.
- * 
- * 
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND WITH ALL FAULTS. ANY EXPRESS OR
- * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
- * PARTICULAR PURPOSE, AND NON-INFRINGEMENT ARE DISCLAIMED AND THE ENTIRE RISK OF SATISFACTORY QUALITY, PERFORMANCE,
- * ACCURACY, AND EFFORT IS WITH LICENSEE. IN NO EVENT SHALL THE COPYRIGHT OWNER, CONTRIBUTORS OR THE UNIVERSITY
- * CORPORATION FOR ADVANCED INTERNET DEVELOPMENT, INC. BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
- * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * All rights reserved Redistribution and use in source and binary forms, with or without modification, are permitted
+ * provided that the following conditions are met: Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer. Redistributions in binary form must reproduce the
+ * above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other
+ * materials provided with the distribution, if any, must include the following acknowledgment: "This product includes
+ * software developed by the University Corporation for Advanced Internet Development <http://www.ucaid.edu> Internet2
+ * Project. Alternately, this acknowledegement may appear in the software itself, if and wherever such third-party
+ * acknowledgments normally appear. Neither the name of Shibboleth nor the names of its contributors, nor Internet2,
+ * nor the University Corporation for Advanced Internet Development, Inc., nor UCAID may be used to endorse or promote
+ * products derived from this software without specific prior written permission. For written permission, please
+ * contact shibboleth@shibboleth.org Products derived from this software may not be called Shibboleth, Internet2,
+ * UCAID, or the University Corporation for Advanced Internet Development, nor may Shibboleth appear in their name,
+ * without prior written permission of the University Corporation for Advanced Internet Development. THIS SOFTWARE IS
+ * PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND WITH ALL FAULTS. ANY EXPRESS OR IMPLIED WARRANTIES,
+ * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE, AND
+ * NON-INFRINGEMENT ARE DISCLAIMED AND THE ENTIRE RISK OF SATISFACTORY QUALITY, PERFORMANCE, ACCURACY, AND EFFORT IS
+ * WITH LICENSEE. IN NO EVENT SHALL THE COPYRIGHT OWNER, CONTRIBUTORS OR THE UNIVERSITY CORPORATION FOR ADVANCED
+ * INTERNET DEVELOPMENT, INC. BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
+ * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR
+ * TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
  */
 
 package edu.internet2.middleware.shibboleth.aa;
@@ -48,10 +37,9 @@ import edu.internet2.middleware.shibboleth.common.ShibbolethOriginConfig;
  */
 public class AAConfig extends ShibbolethOriginConfig {
 
-       private static Logger log = Logger.getLogger(AAConfig.class.getName());
-
-       private String resolverConfig = "/conf/resolver.xml";
-       private boolean passThruErrors = false;
+       private static Logger   log                             = Logger.getLogger(AAConfig.class.getName());
+       private String                  resolverConfig  = "/conf/resolver.xml";
+       private boolean                 passThruErrors  = false;
 
        public AAConfig(Element config) throws ShibbolethConfigurationException {
 
@@ -62,13 +50,15 @@ public class AAConfig extends ShibbolethOriginConfig {
                if (rawResolverConfig != null && !rawResolverConfig.equals("")) {
                        resolverConfig = rawResolverConfig;
                }
-               log.debug("Resolver config file location set to: (" + resolverConfig + ").");
 
                //Global Pass thru error setting
-               String attribute = ((Element) config).getAttribute("passThruErros");
+               String attribute = ((Element) config).getAttribute("passThruErrors");
                if (attribute != null && !attribute.equals("")) {
-                       passThruErrors = Boolean.getBoolean(attribute);
+                       passThruErrors = Boolean.valueOf(attribute).booleanValue();
                }
+
+               log.debug("Global config: (resolverConfig) = (" + getResolverConfigLocation() + ").");
+               log.debug("Global config: (passThruErrors) = (" + passThruErrors() + ").");
        }
 
        public String getResolverConfigLocation() {
index 481c29b..502b469 100644 (file)
@@ -1,49 +1,29 @@
 /*
- * The Shibboleth License, Version 1. Copyright (c) 2002 University Corporation
- * for Advanced Internet Development, Inc. All rights reserved
- * 
- * 
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are met:
- * 
- * Redistributions of source code must retain the above copyright notice, this
- * list of conditions and the following disclaimer.
- * 
- * Redistributions in binary form must reproduce the above copyright notice,
- * this list of conditions and the following disclaimer in the documentation
- * and/or other materials provided with the distribution, if any, must include
- * the following acknowledgment: "This product includes software developed by
- * the University Corporation for Advanced Internet Development
- * <http://www.ucaid.edu> Internet2 Project. Alternately, this acknowledegement
- * may appear in the software itself, if and wherever such third-party
- * acknowledgments normally appear.
- * 
- * Neither the name of Shibboleth nor the names of its contributors, nor
- * Internet2, nor the University Corporation for Advanced Internet Development,
- * Inc., nor UCAID may be used to endorse or promote products derived from this
- * software without specific prior written permission. For written permission,
- * please contact shibboleth@shibboleth.org
- * 
- * Products derived from this software may not be called Shibboleth, Internet2,
- * UCAID, or the University Corporation for Advanced Internet Development, nor
- * may Shibboleth appear in their name, without prior written permission of the
- * University Corporation for Advanced Internet Development.
- * 
- * 
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
- * AND WITH ALL FAULTS. ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
- * PARTICULAR PURPOSE, AND NON-INFRINGEMENT ARE DISCLAIMED AND THE ENTIRE RISK
- * OF SATISFACTORY QUALITY, PERFORMANCE, ACCURACY, AND EFFORT IS WITH LICENSEE.
- * IN NO EVENT SHALL THE COPYRIGHT OWNER, CONTRIBUTORS OR THE UNIVERSITY
- * CORPORATION FOR ADVANCED INTERNET DEVELOPMENT, INC. BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
- * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
- * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
- * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
- * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * The Shibboleth License, Version 1. Copyright (c) 2002 University Corporation for Advanced Internet Development, Inc.
+ * All rights reserved Redistribution and use in source and binary forms, with or without modification, are permitted
+ * provided that the following conditions are met: Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer. Redistributions in binary form must reproduce the
+ * above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other
+ * materials provided with the distribution, if any, must include the following acknowledgment: "This product includes
+ * software developed by the University Corporation for Advanced Internet Development <http://www.ucaid.edu> Internet2
+ * Project. Alternately, this acknowledegement may appear in the software itself, if and wherever such third-party
+ * acknowledgments normally appear. Neither the name of Shibboleth nor the names of its contributors, nor Internet2,
+ * nor the University Corporation for Advanced Internet Development, Inc., nor UCAID may be used to endorse or promote
+ * products derived from this software without specific prior written permission. For written permission, please
+ * contact shibboleth@shibboleth.org Products derived from this software may not be called Shibboleth, Internet2,
+ * UCAID, or the University Corporation for Advanced Internet Development, nor may Shibboleth appear in their name,
+ * without prior written permission of the University Corporation for Advanced Internet Development. THIS SOFTWARE IS
+ * PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND WITH ALL FAULTS. ANY EXPRESS OR IMPLIED WARRANTIES,
+ * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE, AND
+ * NON-INFRINGEMENT ARE DISCLAIMED AND THE ENTIRE RISK OF SATISFACTORY QUALITY, PERFORMANCE, ACCURACY, AND EFFORT IS
+ * WITH LICENSEE. IN NO EVENT SHALL THE COPYRIGHT OWNER, CONTRIBUTORS OR THE UNIVERSITY CORPORATION FOR ADVANCED
+ * INTERNET DEVELOPMENT, INC. BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
+ * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR
+ * TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
  */
+
 package edu.internet2.middleware.shibboleth.aa;
 
 import org.apache.log4j.Logger;
@@ -63,15 +43,16 @@ import edu.internet2.middleware.shibboleth.common.ShibbolethOriginConfig;
  */
 public class AAServiceProviderMapper extends ServiceProviderMapper {
 
-       private static Logger log = Logger.getLogger(AAServiceProviderMapper.class.getName());
-       private AAConfig configuration;
+       private static Logger   log     = Logger.getLogger(AAServiceProviderMapper.class.getName());
+       private AAConfig                configuration;
 
        /**
-         * Constructs a new service provider mapper for the attribute authority.
+        * Constructs a new service provider mapper for the attribute authority.
         * 
-        * @param rawConfig DOM representation of the attribute authority configuration
-        * @param configuration global attribute authority configuration
-         *
+        * @param rawConfig
+        *            DOM representation of the attribute authority configuration
+        * @param configuration
+        *            global attribute authority configuration
         * @throws ServiceProviderMapperException
         *             if the configuration is invalid
         */
@@ -79,8 +60,8 @@ public class AAServiceProviderMapper extends ServiceProviderMapper {
 
                this.configuration = configuration;
 
-               NodeList itemElements =
-                       rawConfig.getElementsByTagNameNS(ShibbolethOriginConfig.originConfigNamespace, "RelyingParty");
+               NodeList itemElements = rawConfig.getElementsByTagNameNS(ShibbolethOriginConfig.originConfigNamespace,
+                               "RelyingParty");
 
                for (int i = 0; i < itemElements.getLength(); i++) {
                        addRelyingParty((Element) itemElements.item(i));
@@ -103,9 +84,9 @@ public class AAServiceProviderMapper extends ServiceProviderMapper {
                }
        }
 
-        /**
-         * Returns the appropriate relying party for the supplied service provider id.
-         */
+       /**
+        * Returns the appropriate relying party for the supplied service provider id.
+        */
        public AARelyingParty getRelyingParty(String providerIdFromTarget) {
                return (AARelyingParty) getRelyingPartyImpl(providerIdFromTarget);
        }
@@ -114,32 +95,32 @@ public class AAServiceProviderMapper extends ServiceProviderMapper {
                return configuration;
        }
 
-        /**
-         * AA-specific relying party implementation.
-         * @author Walter Hoehn
-         */
-       class AARelyingPartyImpl extends BaseRelyingPartyImpl {
+       /**
+        * AA-specific relying party implementation.
+        * 
+        * @author Walter Hoehn
+        */
+       class AARelyingPartyImpl extends BaseRelyingPartyImpl implements AARelyingParty {
 
-               private AAConfig aaConfig;
-               private boolean overridenPassThruErrors;
-               private boolean passThruIsOverriden;
+               private AAConfig        aaConfig;
+               private boolean         overridenPassThruErrors = false;
+               private boolean         passThruIsOverriden             = false;
 
                public AARelyingPartyImpl(Element partyConfig, AAConfig globalConfig) throws ServiceProviderMapperException {
                        super(partyConfig);
 
                        aaConfig = globalConfig;
 
-                       String attribute = ((Element) partyConfig).getAttribute("passThruErros");
+                       String attribute = ((Element) partyConfig).getAttribute("passThruErrors");
                        if (attribute != null && !attribute.equals("")) {
                                log.debug("Overriding passThruErrors for Relying Pary (" + name + ") with (" + attribute + ").");
-                               overridenPassThruErrors = Boolean.getBoolean(attribute);
+                               overridenPassThruErrors = Boolean.valueOf(attribute).booleanValue();
                                passThruIsOverriden = true;
                        }
 
-                       identityProvider =
-                               new RelyingPartyIdentityProvider(
-                                       overridenOriginProviderId != null ? overridenOriginProviderId : configuration.getProviderId(),
-                                       null);
+                       identityProvider = new RelyingPartyIdentityProvider(overridenOriginProviderId != null
+                                       ? overridenOriginProviderId
+                                       : configuration.getProviderId(), null);
                }
 
                public boolean passThruErrors() {
index 1d514d7..f60bdc6 100755 (executable)
@@ -250,8 +250,6 @@ public class AAServlet extends HttpServlet {
                                }
                                log.info("Request is for principal (" + principal.getName() + ").");
 
-                               //TODO Do something about these silly passthru errors
-
                        } catch (InvalidNameIdentifierException invalidNameE) {
                                log.info("Could not associate the request subject with a principal: " + invalidNameE);
                                try {