Cleanup service interface, make it easier and less error prone when creating new...
[java-idp.git] / src / edu / internet2 / middleware / shibboleth / idp / profile / IdPProfileHandlerManager.java
index a32863d..0a20620 100644 (file)
 package edu.internet2.middleware.shibboleth.idp.profile;
 
 import java.util.HashMap;
-import java.util.List;
 import java.util.Map;
-import java.util.Timer;
 import java.util.concurrent.locks.Lock;
 
 import javax.servlet.ServletRequest;
 import javax.servlet.http.HttpServletRequest;
 
-import org.opensaml.util.resource.Resource;
-import org.opensaml.xml.util.Pair;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.context.ApplicationContext;
@@ -37,7 +33,6 @@ import edu.internet2.middleware.shibboleth.common.profile.ProfileHandler;
 import edu.internet2.middleware.shibboleth.common.profile.ProfileHandlerManager;
 import edu.internet2.middleware.shibboleth.common.profile.provider.AbstractRequestURIMappedProfileHandler;
 import edu.internet2.middleware.shibboleth.idp.authn.LoginHandler;
-import edu.internet2.middleware.shibboleth.idp.authn.LoginContext;
 
 /**
  * Implementation of a {@link ProfileHandlerManager} that maps the request path, without the servlet context, to a
@@ -57,27 +52,9 @@ public class IdPProfileHandlerManager extends BaseReloadableService implements P
     /** Map of authentication methods to login handlers. */
     private Map<String, LoginHandler> loginHandlers;
 
-    /**
-     * Constructor. Configuration resources are not monitored for changes.
-     * 
-     * @param configurations configuration resources for this service
-     */
-    public IdPProfileHandlerManager(List<Resource> configurations) {
-        super(configurations);
-        profileHandlers = new HashMap<String, AbstractRequestURIMappedProfileHandler>();
-        loginHandlers = new HashMap<String, LoginHandler>();
-    }
-
-    /**
-     * Constructor.
-     * 
-     * @param timer timer resource polling tasks are scheduled with
-     * @param configurations configuration resources for this service
-     * @param pollingFrequency the frequency, in milliseconds, to poll the policy resources for changes, must be greater
-     *            than zero
-     */
-    public IdPProfileHandlerManager(List<Resource> configurations, Timer timer, long pollingFrequency) {
-        super(timer, configurations, pollingFrequency);
+    /** Constructor. */
+    public IdPProfileHandlerManager() {
+        super();
         profileHandlers = new HashMap<String, AbstractRequestURIMappedProfileHandler>();
         loginHandlers = new HashMap<String, LoginHandler>();
     }
@@ -139,7 +116,7 @@ public class IdPProfileHandlerManager extends BaseReloadableService implements P
     }
 
     /** {@inheritDoc} */
-    protected void newContextCreated(ApplicationContext newServiceContext) {
+    protected void onNewContextCreated(ApplicationContext newServiceContext) {
         log.debug("{}: Loading new configuration into service", getId());
         Lock writeLock = getReadWriteLock().writeLock();
         writeLock.lock();