Fixed a few bugs caused by the conversion away from Properties.
authorwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Wed, 4 Feb 2004 06:00:12 +0000 (06:00 +0000)
committerwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Wed, 4 Feb 2004 06:00:12 +0000 (06:00 +0000)
git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/trunk@874 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

src/edu/internet2/middleware/shibboleth/aa/AAServiceProviderMapper.java
src/edu/internet2/middleware/shibboleth/common/ServiceProviderMapper.java
src/edu/internet2/middleware/shibboleth/common/ShibbolethOriginConfig.java
src/edu/internet2/middleware/shibboleth/hs/HSServiceProviderMapper.java
src/edu/internet2/middleware/shibboleth/hs/HandleServlet.java

index 8f2cfb8..00a7083 100644 (file)
@@ -92,6 +92,7 @@ public class AAServiceProviderMapper extends ServiceProviderMapper {
        }
 
        public AARelyingParty getRelyingParty(String providerIdFromTarget) {
+               //TODO hmmm...
                return (AARelyingParty) getRelyingPartyImpl(providerIdFromTarget);
        }
 
@@ -113,7 +114,10 @@ public class AAServiceProviderMapper extends ServiceProviderMapper {
                                passThruIsOverriden = true;
                        }
 
-                       identityProvider = new RelyingPartyIdentityProvider(overridenOriginProviderId, null);
+                       identityProvider =
+                               new RelyingPartyIdentityProvider(
+                                       overridenOriginProviderId != null ? overridenOriginProviderId : configuration.getProviderId(),
+                                       null);
                }
 
                public boolean passThruErrors() {
index 83aa5bc..dade6b7 100644 (file)
@@ -65,7 +65,6 @@ public abstract class ServiceProviderMapper {
 
        protected void verifyDefaultParty(ShibbolethOriginConfig configuration) throws ServiceProviderMapperException {
                //Verify we have a proper default party
-               System.err.println("Walter: " + configuration.getDefaultRelyingPartyName());
                String defaultParty = configuration.getDefaultRelyingPartyName();
                if (defaultParty == null || defaultParty.equals("")) {
                        if (relyingParties.size() != 1) {
@@ -170,7 +169,7 @@ public abstract class ServiceProviderMapper {
                public IdentityProvider getIdentityProvider() {
                        return identityProvider;
                }
-
+               
                protected class RelyingPartyIdentityProvider implements IdentityProvider {
 
                        private String providerId;
index 6383429..613a46d 100644 (file)
@@ -71,14 +71,14 @@ public abstract class ShibbolethOriginConfig {
                log.debug("Loading global configuration properties.");
 
                //Global providerId
-               String providerId = ((Element) config).getAttribute("providerId");
+               providerId = ((Element) config).getAttribute("providerId");
                if (providerId == null || providerId.equals("")) {
                        log.error("Global providerId not set.  Add a (providerId) attribute to <ShibbolethOriginConfig>.");
                        throw new ShibbolethConfigurationException("Required configuration not specified.");
                }
 
                //Default Relying Party
-               String defaultRelyingPartyName = ((Element) config).getAttribute("defaultRelyingParty");
+               defaultRelyingPartyName = ((Element) config).getAttribute("defaultRelyingParty");
                if (defaultRelyingPartyName == null || defaultRelyingPartyName.equals("")) {
                        log.error("Default Relying Party not set.  Add a (defaultRelyingParty) attribute to <ShibbolethOriginConfig>.");
                        throw new ShibbolethConfigurationException("Required configuration not specified.");
index ae5c037..8f0bbc4 100644 (file)
@@ -1,38 +1,48 @@
 /*
- * The Shibboleth License, Version 1. Copyright (c) 2002 University Corporation for Advanced Internet Development, Inc.
- * All rights reserved
+ * 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:
+ * 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 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.
+ * 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
+ * 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
+ * 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.
+ * 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.hs;
 
@@ -68,7 +78,7 @@ public class HSServiceProviderMapper extends ServiceProviderMapper {
                Credentials credentials,
                HSNameMapper nameMapper)
                throws ServiceProviderMapperException {
-System.err.println("Walter 2: " + configuration.getDefaultRelyingPartyName());
+
                this.configuration = configuration;
                this.credentials = credentials;
                this.nameMapper = nameMapper;
@@ -108,7 +118,7 @@ System.err.println("Walter 2: " + configuration.getDefaultRelyingPartyName());
 
                return (HSRelyingParty) getRelyingPartyImpl(providerIdFromTarget);
        }
-       class HSRelyingPartyImpl extends BaseRelyingPartyImpl {
+       class HSRelyingPartyImpl extends BaseRelyingPartyImpl implements HSRelyingParty {
 
                private URL overridenAAUrl;
                private URI overridenDefaultAuthMethod;
@@ -200,7 +210,10 @@ System.err.println("Walter 2: " + configuration.getDefaultRelyingPartyName());
                                }
                        }
 
-                       identityProvider = new RelyingPartyIdentityProvider(overridenOriginProviderId, credential);
+                       identityProvider =
+                               new RelyingPartyIdentityProvider(
+                                       overridenOriginProviderId != null ? overridenOriginProviderId : configuration.getProviderId(),
+                                       credential);
                }
 
                public boolean isLegacyProvider() {
index 3c08ddb..aa89058 100644 (file)
@@ -109,7 +109,6 @@ public class HandleServlet extends OriginComponent {
 
                //Load global configuration properties
                configuration = new HSConfig(parser.getDocument().getDocumentElement());
-               System.out.println("Walter 3: " + configuration.getDefaultRelyingPartyName());
 
                //Load signing credentials
                NodeList itemElements =