Skip to content

Commit c39d39b

Browse files
ClaudioConsolmagnojzheaux
authored andcommitted
Use 'md:' prefix in EntityDescriptor XML
Create the EntityDescriptor object with EntityDescriptor.DEFAULT_ELEMENT_NAME instead of EntityDescriptor.ELEMENT_QNAME. That ensures the EntityDescriptor tag is marshalled to xml with the 'md:' prefix, consistent with all other metadata tags. Closes #11283
1 parent 2925850 commit c39d39b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

saml2/saml2-service-provider/src/main/java/org/springframework/security/saml2/provider/service/metadata/OpenSamlMetadataResolver.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ public OpenSamlMetadataResolver() {
7575

7676
@Override
7777
public String resolve(RelyingPartyRegistration relyingPartyRegistration) {
78-
EntityDescriptor entityDescriptor = build(EntityDescriptor.ELEMENT_QNAME);
78+
EntityDescriptor entityDescriptor = build(EntityDescriptor.DEFAULT_ELEMENT_NAME);
7979
entityDescriptor.setEntityID(relyingPartyRegistration.getEntityId());
8080
SPSSODescriptor spSsoDescriptor = buildSpSsoDescriptor(relyingPartyRegistration);
8181
entityDescriptor.getRoleDescriptors(SPSSODescriptor.DEFAULT_ELEMENT_NAME).add(spSsoDescriptor);

saml2/saml2-service-provider/src/test/java/org/springframework/security/saml2/provider/service/metadata/OpenSamlMetadataResolverTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public void resolveWhenRelyingPartyThenMetadataMatches() {
3636
.assertionConsumerServiceBinding(Saml2MessageBinding.REDIRECT).build();
3737
OpenSamlMetadataResolver openSamlMetadataResolver = new OpenSamlMetadataResolver();
3838
String metadata = openSamlMetadataResolver.resolve(relyingPartyRegistration);
39-
assertThat(metadata).contains("<EntityDescriptor").contains("entityID=\"rp-entity-id\"")
39+
assertThat(metadata).contains("<md:EntityDescriptor").contains("entityID=\"rp-entity-id\"")
4040
.contains("<md:KeyDescriptor use=\"signing\">").contains("<md:KeyDescriptor use=\"encryption\">")
4141
.contains("<ds:X509Certificate>MIICgTCCAeoCCQCuVzyqFgMSyDANBgkqhkiG9w0BAQsFADCBhDELMAkGA1UEBh")
4242
.contains("Binding=\"urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect\"")
@@ -52,7 +52,7 @@ public void resolveWhenRelyingPartyNoCredentialsThenMetadataMatches() {
5252
.build();
5353
OpenSamlMetadataResolver openSamlMetadataResolver = new OpenSamlMetadataResolver();
5454
String metadata = openSamlMetadataResolver.resolve(relyingPartyRegistration);
55-
assertThat(metadata).contains("<EntityDescriptor").contains("entityID=\"rp-entity-id\"")
55+
assertThat(metadata).contains("<md:EntityDescriptor").contains("entityID=\"rp-entity-id\"")
5656
.doesNotContain("<md:KeyDescriptor use=\"signing\">")
5757
.doesNotContain("<md:KeyDescriptor use=\"encryption\">")
5858
.contains("Binding=\"urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST\"")
@@ -86,7 +86,7 @@ public void resolveWhenEntityDescriptorCustomizerThenUses() {
8686
openSamlMetadataResolver.setEntityDescriptorCustomizer(
8787
(parameters) -> parameters.getEntityDescriptor().setEntityID("overriddenEntityId"));
8888
String metadata = openSamlMetadataResolver.resolve(relyingPartyRegistration);
89-
assertThat(metadata).contains("<EntityDescriptor").contains("entityID=\"overriddenEntityId\"");
89+
assertThat(metadata).contains("<md:EntityDescriptor").contains("entityID=\"overriddenEntityId\"");
9090
}
9191

9292
}

0 commit comments

Comments
 (0)