diff --git a/src/main/java/org/opensearch/security/action/onbehalf/CreateOnBehalfOfTokenAction.java b/src/main/java/org/opensearch/security/action/onbehalf/CreateOnBehalfOfTokenAction.java index c5862a1537..3e67ae859c 100644 --- a/src/main/java/org/opensearch/security/action/onbehalf/CreateOnBehalfOfTokenAction.java +++ b/src/main/java/org/opensearch/security/action/onbehalf/CreateOnBehalfOfTokenAction.java @@ -61,16 +61,13 @@ public void onDynamicConfigModelChanged(DynamicConfigModel dcm) { this.dcm = dcm; Settings settings = dcm.getDynamicOnBehalfOfSettings(); - if (settings != null) { - Boolean enabled = Boolean.parseBoolean(settings.get("enabled")); - String signingKey = settings.get("signing_key"); - String encryptionKey = settings.get("encryption_key"); - - if (!Boolean.FALSE.equals(enabled) && signingKey != null && encryptionKey != null) { - this.vendor = new JwtVendor(settings, Optional.empty()); - } else { - this.vendor = null; - } + + Boolean enabled = Boolean.parseBoolean(settings.get("enabled")); + String signingKey = settings.get("signing_key"); + String encryptionKey = settings.get("encryption_key"); + + if (!Boolean.FALSE.equals(enabled) && signingKey != null && encryptionKey != null) { + this.vendor = new JwtVendor(settings, Optional.empty()); } else { this.vendor = null; }