[java-plugin-shibd] branch main updated: Deprecate old target tracking feature and add passive flag to StateData.

[Codeberg]

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

codeberg pushed a commit to branch main
in repository java-plugin-shibd.

View the commit online:
https://codeberg.org/Shibboleth/java-plugin-shibd/commit/8ad2ded400535c930870beddd82fa19bc1aa41ad

The following commit(s) were added to refs/heads/main by this push:
     new 8ad2ded  Deprecate old target tracking feature and add passive flag to StateData.
8ad2ded is described below

commit 8ad2ded400535c930870beddd82fa19bc1aa41ad
Author: Scott Cantor <scott at restingparrotsoftware.com>
AuthorDate: Mon Apr 27 11:39:12 2026 -0400

    Deprecate old target tracking feature and add passive flag to StateData.
---
 .../shibboleth/sp/context/AgentRequestContext.java |  6 +++++
 .../sp/profile/PreserveStateDataAction.java        |  2 +-
 .../java/net/shibboleth/sp/state/StateData.java    | 28 +++++++++++++++++++++-
 3 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/sp-server-api/src/main/java/net/shibboleth/sp/context/AgentRequestContext.java b/sp-server-api/src/main/java/net/shibboleth/sp/context/AgentRequestContext.java
index dcdf3a0..6bee563 100644
--- a/sp-server-api/src/main/java/net/shibboleth/sp/context/AgentRequestContext.java
+++ b/sp-server-api/src/main/java/net/shibboleth/sp/context/AgentRequestContext.java
@@ -105,7 +105,10 @@ public class AgentRequestContext extends BaseContext {
      * have access to it in the event of a problem.</p>
      * 
      * @return target URL
+     * 
+     * @deprecated
      */
+    @Deprecated
     @Nullable public byte[] getTargetURL() {
         return targetURL;
     }
@@ -116,7 +119,10 @@ public class AgentRequestContext extends BaseContext {
      * @param url url to set
      * 
      * @return this context
+     * 
+     * @deprecated
      */
+    @Deprecated
     @Nonnull public AgentRequestContext setTargetURL(@Nullable final byte[] url) {
         targetURL = url;
         
diff --git a/sp-server-api/src/main/java/net/shibboleth/sp/profile/PreserveStateDataAction.java b/sp-server-api/src/main/java/net/shibboleth/sp/profile/PreserveStateDataAction.java
index f381812..d0fcd0c 100644
--- a/sp-server-api/src/main/java/net/shibboleth/sp/profile/PreserveStateDataAction.java
+++ b/sp-server-api/src/main/java/net/shibboleth/sp/profile/PreserveStateDataAction.java
@@ -60,7 +60,7 @@ public class PreserveStateDataAction extends AbstractApplicationAction {
     /** Class logger. */
     @Nonnull private Logger log = LoggerFactory.getLogger(PreserveStateDataAction.class);
 
-    /** Strategy used to create the {@link StateDataContext} to populate. */
+    /** Strategy used to create the {@link StateDataContext} to presrve. */
     @Nonnull private Function<ProfileRequestContext,StateDataContext> stateDataContextLookupStrategy;
     
     /** Whether an error constructing a state token is fatal. */
diff --git a/sp-server-api/src/main/java/net/shibboleth/sp/state/StateData.java b/sp-server-api/src/main/java/net/shibboleth/sp/state/StateData.java
index cc3e063..320f5f8 100644
--- a/sp-server-api/src/main/java/net/shibboleth/sp/state/StateData.java
+++ b/sp-server-api/src/main/java/net/shibboleth/sp/state/StateData.java
@@ -33,7 +33,6 @@ import com.google.common.base.MoreObjects;
 import net.shibboleth.shared.annotation.constraint.Live;
 import net.shibboleth.shared.annotation.constraint.NotLive;
 import net.shibboleth.shared.annotation.constraint.Unmodifiable;
-import net.shibboleth.shared.collection.CollectionSupport;
 import net.shibboleth.shared.primitive.StringSupport;
 
 /**
@@ -61,6 +60,9 @@ public class StateData {
     /** The expected issuer of the authentication response. */
     @Nullable private String authenticatingAuthority;
     
+    /** Set when request is passive. */
+    @Nullable private Boolean passive;
+    
     /** The time at which a request was issued. */
     @Nullable private Instant requestTime;
     
@@ -149,6 +151,28 @@ public class StateData {
         return this;
     }
     
+    /**
+     * Get the passive request indicator.
+     * 
+     * @return passive indicator
+     */
+    @JsonProperty("passive")
+    @Nullable public Boolean getPassive() {
+        return passive;
+    }
+    
+    /**
+     * Set the passive request indicator.
+     * 
+     * @param flag indicator
+     * 
+     * @return the updated object
+     */
+    @Nonnull public StateData setPassive(@Nullable final Boolean flag) {
+        passive = flag;
+        return this;
+    }
+    
     /**
      * Get the time at which the request was issued.
      * 
@@ -291,6 +315,7 @@ public class StateData {
         final StateData other = (StateData) obj;
         return Objects.equals(acrs, other.acrs)
                 && Objects.equals(authenticatingAuthority, other.authenticatingAuthority)
+                && Objects.equals(passive, other.passive)
                 && Objects.equals(issuer, other.issuer)
                 && Objects.equals(requestTime, other.requestTime)
                 && Objects.equals(resource, other.resource)
@@ -305,6 +330,7 @@ public class StateData {
                 .add("clientAddress", clientAddress)
                 .add("issuer", issuer)
                 .add("authenticatingAuthority", authenticatingAuthority)
+                .add("passive", passive)
                 .add("requestTime", requestTime)
                 .add("acrs", acrs)
                 .add("reponseLocation", responseLocation)

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