Skip to content

Commit d3e0ff4

Browse files
committed
Only include extensions in JSON if populated
1 parent 975d8b9 commit d3e0ff4

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

x-pack/plugin/identity-provider/src/main/java/org/elasticsearch/xpack/idp/saml/sp/SamlServiceProviderDocument.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -538,7 +538,9 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws
538538
builder.field(Fields.Attributes.EMAIL.getPreferredName(), attributeNames.email);
539539
builder.field(Fields.Attributes.NAME.getPreferredName(), attributeNames.name);
540540
builder.field(Fields.Attributes.ROLES.getPreferredName(), attributeNames.roles);
541-
builder.field(Fields.Attributes.EXTENSIONS.getPreferredName(), attributeNames.extensions);
541+
if (attributeNames.extensions != null && attributeNames.extensions.isEmpty() == false) {
542+
builder.field(Fields.Attributes.EXTENSIONS.getPreferredName(), attributeNames.extensions);
543+
}
542544
builder.endObject();
543545

544546
builder.startObject(Fields.CERTIFICATES.getPreferredName());

0 commit comments

Comments
 (0)