github
/
OpenID-Connect-Java-Spring-Server
mirror of https://github.com/mitreid-connect/OpenID-Connect-Java-Spring-Server
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix: 🐛 Remove RelayState from SAML details in SavedUserAuth

pull/1580/head
Dominik Frantisek Bucik 2022-04-04 08:45:23 +02:00
parent d0954ada62
commit 0f73d88236
No known key found for this signature in database
GPG Key ID: 25014C8DB2E7E62D
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
perun-oidc-server/src/main/java/cz/muni/ics/oauth2/model/SamlAuthenticationDetails.java
View File

@ -29,7 +29,6 @@ import org.springframework.util.StringUtils;
@AllArgsConstructor @AllArgsConstructor
public class SamlAuthenticationDetails { public class SamlAuthenticationDetails {
public static final String RELAY_STATE = "relayState";
public static final String LOCAL_ENTITY_ID = "localEntityId"; public static final String LOCAL_ENTITY_ID = "localEntityId";
public static final String REMOTE_ENTITY_ID = "remoteEntityId"; public static final String REMOTE_ENTITY_ID = "remoteEntityId";
public static final String ATTRIBUTES = "attributes"; public static final String ATTRIBUTES = "attributes";
@ -37,14 +36,12 @@ public class SamlAuthenticationDetails {
public static final String AUTHN_CONTEXT_CLASS_REF = "authnContextClassRef"; public static final String AUTHN_CONTEXT_CLASS_REF = "authnContextClassRef";
public static final String AUTHENTICATING_AUTHORITIES = "authenticatingAuthorities"; public static final String AUTHENTICATING_AUTHORITIES = "authenticatingAuthorities";
private String relayState;
private String remoteEntityID; private String remoteEntityID;
private String localEntityID; private String localEntityID;
private Map<String, String[]> attributes; private Map<String, String[]> attributes;
private List<AuthenticationStatement> authnStatements; private List<AuthenticationStatement> authnStatements;
public SamlAuthenticationDetails(SAMLCredential samlCredential) { public SamlAuthenticationDetails(SAMLCredential samlCredential) {
this.relayState = samlCredential.getRelayState();
this.remoteEntityID = samlCredential.getRemoteEntityID(); this.remoteEntityID = samlCredential.getRemoteEntityID();
this.localEntityID = samlCredential.getLocalEntityID(); this.localEntityID = samlCredential.getLocalEntityID();
this.attributes = processAttributes(samlCredential); this.attributes = processAttributes(samlCredential);
@ -104,7 +101,6 @@ public class SamlAuthenticationDetails {
} }
JsonObject json = (JsonObject) JsonParser.parseString(strJson); JsonObject json = (JsonObject) JsonParser.parseString(strJson);
SamlAuthenticationDetails details = new SamlAuthenticationDetails(); SamlAuthenticationDetails details = new SamlAuthenticationDetails();
details.setRelayState(getStringOrNull(json.get(RELAY_STATE)));
details.setRemoteEntityID(getStringOrNull(json.get(REMOTE_ENTITY_ID))); details.setRemoteEntityID(getStringOrNull(json.get(REMOTE_ENTITY_ID)));
details.setLocalEntityID(getStringOrNull(json.get(LOCAL_ENTITY_ID))); details.setLocalEntityID(getStringOrNull(json.get(LOCAL_ENTITY_ID)));
@ -144,7 +140,6 @@ public class SamlAuthenticationDetails {
return null; return null;
} }
JsonObject object = new JsonObject(); JsonObject object = new JsonObject();
addStringOrNull(object, RELAY_STATE, o.getRelayState());
addStringOrNull(object, LOCAL_ENTITY_ID, o.getLocalEntityID()); addStringOrNull(object, LOCAL_ENTITY_ID, o.getLocalEntityID());
addStringOrNull(object, REMOTE_ENTITY_ID, o.getRemoteEntityID()); addStringOrNull(object, REMOTE_ENTITY_ID, o.getRemoteEntityID());
JsonObject attrs = new JsonObject(); JsonObject attrs = new JsonObject();