libre.sh/keycloak-scim
3
0
Fork 0
Code Issues 15 Pull requests Releases Packages Activity Actions

KEYCLOAK-5842 Fix NPE in admin-cli ParseUtil.mergeAttributes

Browse source
This commit is contained in:
Marko Strukelj 2017-11-20 16:06:27 +01:00
parent 8e53ccf5ab
commit 8f82440103
1 changed files with 13 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
integration/client-cli/admin-cli/src/main/java/org/keycloak/client/admin/cli/util/ParseUtil.java
View file

@ -74,7 +74,7 @@ public class ParseUtil {
}
public static <T> CmdStdinContext<JsonNode> mergeAttributes(CmdStdinContext<JsonNode> ctx, ObjectNode newObject, List<AttributeOperation> attrs) {
String content = ctx.getContent();
JsonNode node = ctx.getResult();
if (node != null && !node.isObject()) {
throw new RuntimeException("Not a JSON object: " + node);
@ -83,28 +83,24 @@ public class ParseUtil {
try {
if (result == null) {
try {
result = newObject;
} catch (Throwable e) {
throw new RuntimeException("Failed to instantiate object: " + e.getMessage(), e);
}
}
if (result != null) {
if (result == null) {
throw new RuntimeException("Failed to set attribute(s) - no target object");
}
try {
setAttributes(result, attrs);
} catch (AttributeException e) {
throw new RuntimeException("Failed to set attribute '" + e.getAttributeName() + "' on document type '" + result.getClass().getName() + "'", e);
}
content = JsonSerialization.writeValueAsString(result);
} else {
throw new RuntimeException("Setting attributes is not supported for type: " + result.getClass().getName());
}
ctx.setContent(JsonSerialization.writeValueAsString(result));
} catch (IOException e) {
throw new RuntimeException("Failed to merge set attributes with configuration from file", e);
throw new RuntimeException("Failed to merge attributes with configuration from file", e);
}
ctx.setContent(content);
ctx.setResult(result);
return ctx;
}