From fe2621a5039b34a0074e8cc88026a1d57c7dddb6 Mon Sep 17 00:00:00 2001
From: Michael Simon <simon@kit.edu>
Date: Tue, 5 Nov 2024 07:14:21 +0100
Subject: [PATCH] NO_STORY change logging statement for missing methods
We assume that not all methods must be defined.
---
.../webreg/service/oidc/OidcOpStaticLoginProcessor.java | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/bwreg-service/src/main/java/edu/kit/scc/webreg/service/oidc/OidcOpStaticLoginProcessor.java b/bwreg-service/src/main/java/edu/kit/scc/webreg/service/oidc/OidcOpStaticLoginProcessor.java
index e8a0c639..dfa065d5 100644
--- a/bwreg-service/src/main/java/edu/kit/scc/webreg/service/oidc/OidcOpStaticLoginProcessor.java
+++ b/bwreg-service/src/main/java/edu/kit/scc/webreg/service/oidc/OidcOpStaticLoginProcessor.java
@@ -366,7 +366,9 @@ public class OidcOpStaticLoginProcessor extends AbstractOidcOpLoginProcessor {
invocable.invokeFunction("resolveAttributes", scriptingEnv, attributeBuilder, attributeResolver,
attributeRelease, identity, user, registry, logger, flowState, consumerConfig, opConfig);
- } catch (NoSuchMethodException | ScriptException e) {
+ } catch (NoSuchMethodException e) {
+ logger.info("Skipping resolveAttributes method. It's not defined.");
+ } catch (ScriptException e) {
logger.warn("Script execution failed. Continue with other scripts.", e);
}
} else {
@@ -397,7 +399,9 @@ public class OidcOpStaticLoginProcessor extends AbstractOidcOpLoginProcessor {
invocable.invokeFunction(methodName, scriptingEnv, claimsBuilder, user, registry,
serviceOidcClient.getService(), logger, identity);
- } catch (NoSuchMethodException | ScriptException e) {
+ } catch (NoSuchMethodException e) {
+ logger.info("Skipping " + methodName + " method. It's not defined.");
+ } catch (ScriptException e) {
logger.warn("Script execution failed. Continue with other scripts.", e);
}
} else {
--
GitLab