Refactor: centralized dynamic config reset
Moved the initial reset of dynamic configuration to occur a single time
during `GerritClient` initialization, rather than upon each component
client class initialization.
Jira-Id: IT-103
Change-Id: Ieb9e0b95cd5c77a2234fb35f1b9645aacae6fd94
Signed-off-by: Patrizio <patrizio.gelosi@amarulasolutions.com>
diff --git a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClient.java b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClient.java
index a93d10d..ccbcdaa 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClient.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClient.java
@@ -39,6 +39,7 @@
public void initialize(Configuration config, String fullChangeId) {
log.debug("Initializing client instances for change: {}", fullChangeId);
+ config.resetDynamicConfiguration();
for (GerritClientType clientTypes : clientMap.keySet()) {
updateGerritClient(clientTypes, fullChangeId, config);
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClientBase.java b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClientBase.java
index f92410a..057be7b 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClientBase.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/client/gerrit/GerritClientBase.java
@@ -30,7 +30,6 @@
public GerritClientBase(Configuration config) {
this.config = config;
- config.resetDynamicConfiguration();
}
protected String generateBasicAuth(String username, String password) {
