[java-identity-provider] 03/05: IDP-1793 Use Suppliers for HttpRequest/Response

[Rod Widdowson]

This is an automated email from the git hooks/post-receive script.

rdw pushed a commit to branch main
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=6891f0ae6d898dc1973618f8f1460f70749458a1

commit 6891f0ae6d898dc1973618f8f1460f70749458a1
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Thu Jul 28 20:15:03 2022 +0100

    IDP-1793 Use Suppliers for HttpRequest/Response
    
    https://shibboleth.atlassian.net/browse/IDP-1793
    
    Inject the Supplier directly into the RelyingPartyUIContext
---
 .../impl/InitializeAdministrativeProfileContextTree.java  | 15 ++-------------
 .../net/shibboleth/idp/ui/impl/SetRPUIInformation.java    | 14 ++------------
 2 files changed, 4 insertions(+), 25 deletions(-)

diff --git a/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/InitializeAdministrativeProfileContextTree.java b/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/InitializeAdministrativeProfileContextTree.java
index 8298fa637..3cbbeb067 100644
--- a/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/InitializeAdministrativeProfileContextTree.java
+++ b/idp-admin-impl/src/main/java/net/shibboleth/idp/admin/impl/InitializeAdministrativeProfileContextTree.java
@@ -18,7 +18,6 @@
 package net.shibboleth.idp.admin.impl;
 
 import java.util.List;
-import java.util.function.Supplier;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -28,7 +27,6 @@ import org.opensaml.profile.context.ProfileRequestContext;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import jakarta.servlet.http.HttpServletRequest;
 import net.shibboleth.ext.spring.util.SpringSupport;
 import net.shibboleth.idp.admin.AdministrativeFlowDescriptor;
 import net.shibboleth.idp.profile.AbstractProfileAction;
@@ -36,7 +34,6 @@ import net.shibboleth.idp.profile.IdPEventIds;
 import net.shibboleth.idp.profile.context.RelyingPartyContext;
 import net.shibboleth.idp.ui.context.RelyingPartyUIContext;
 import net.shibboleth.utilities.java.support.annotation.constraint.NonnullElements;
-import net.shibboleth.utilities.java.support.net.ThreadLocalHttpServletRequestProxy;
 
 /**
  * An action that processes settings from a supplied {@link AdministrativeFlowDescriptor} to prepare
@@ -123,16 +120,8 @@ public class InitializeAdministrativeProfileContextTree extends AbstractProfileA
         
         final RelyingPartyUIContext uiCtx = rpCtx.getSubcontext(RelyingPartyUIContext.class, true);
         uiCtx.setRPUInfo(flowDescriptor.getUIInfo());
-        final HttpServletRequest request = getHttpServletRequest();
-        uiCtx.setBrowserLanguageRanges(SpringSupport.getLanguageRange(request));
-        if (request instanceof ThreadLocalHttpServletRequestProxy) {
-            // The request is delegated so can be put into a Supplier
-            uiCtx.setRequestSupplier(new Supplier<HttpServletRequest>() {
-                public HttpServletRequest get() {
-                    return request;
-                }
-            });
-        }
+        uiCtx.setBrowserLanguageRanges(SpringSupport.getLanguageRange(getHttpServletRequest()));
+        uiCtx.setRequestSupplier(getHttpServletRequestSupplier());
         
         if (null != fallbackLanguages) {
             uiCtx.setFallbackLanguages(fallbackLanguages);
diff --git a/idp-ui/src/main/java/net/shibboleth/idp/ui/impl/SetRPUIInformation.java b/idp-ui/src/main/java/net/shibboleth/idp/ui/impl/SetRPUIInformation.java
index 5b5834a58..8fc9500da 100644
--- a/idp-ui/src/main/java/net/shibboleth/idp/ui/impl/SetRPUIInformation.java
+++ b/idp-ui/src/main/java/net/shibboleth/idp/ui/impl/SetRPUIInformation.java
@@ -19,7 +19,6 @@ package net.shibboleth.idp.ui.impl;
 
 import java.util.List;
 import java.util.function.Function;
-import java.util.function.Supplier;
 
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -48,7 +47,6 @@ import net.shibboleth.idp.saml.profile.context.navigate.SAMLMetadataContextLooku
 import net.shibboleth.idp.ui.context.RelyingPartyUIContext;
 import net.shibboleth.utilities.java.support.annotation.constraint.NonnullElements;
 import net.shibboleth.utilities.java.support.logic.Constraint;
-import net.shibboleth.utilities.java.support.net.ThreadLocalHttpServletRequestProxy;
 import net.shibboleth.utilities.java.support.primitive.StringSupport;
 
 /**
@@ -212,16 +210,8 @@ public class SetRPUIInformation extends AbstractProfileAction {
         rpUIContext.setRPSPSSODescriptor(spSSODescriptor);
         rpUIContext.setRPAttributeConsumingService(acsDesriptor);
         rpUIContext.setRPUInfo(getRPUInfo());
-        final HttpServletRequest request = getHttpServletRequest();
-        rpUIContext.setBrowserLanguageRanges(SpringSupport.getLanguageRange(request));
-        if (request instanceof ThreadLocalHttpServletRequestProxy) {
-            // The request is delegated so can be put into a Supplier
-            rpUIContext.setRequestSupplier(new Supplier<HttpServletRequest>() {
-                public HttpServletRequest get() {
-                    return request;
-                }
-            });
-        }
+        rpUIContext.setBrowserLanguageRanges(SpringSupport.getLanguageRange(getHttpServletRequest()));
+        rpUIContext.setRequestSupplier(getHttpServletRequestSupplier());
    }
 
 }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.