From e58a5ed31daa2aee5fd099e6228cd55cc20af0d4 Mon Sep 17 00:00:00 2001 From: Robert Manning Date: Tue, 13 Feb 2024 16:00:34 -0500 Subject: [PATCH] fix: move config validation to startup stage so bad configs will break the component --- .../greengrass/clientdevices/auth/ClientDevicesAuthService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/aws/greengrass/clientdevices/auth/ClientDevicesAuthService.java b/src/main/java/com/aws/greengrass/clientdevices/auth/ClientDevicesAuthService.java index 275923517..6b01f02ac 100644 --- a/src/main/java/com/aws/greengrass/clientdevices/auth/ClientDevicesAuthService.java +++ b/src/main/java/com/aws/greengrass/clientdevices/auth/ClientDevicesAuthService.java @@ -108,7 +108,6 @@ protected void install() throws InterruptedException { context.get(CertificateManager.class).updateCertificatesConfiguration(new CertificatesConfig(getConfig())); initializeInfrastructure(); initializeHandlers(); - subscribeToConfigChanges(); } private int getValidCloudCallQueueSize(Topics topics) { @@ -214,6 +213,7 @@ private void configChangeHandler(WhatHappened whatHappened, Node node) { @Override protected void startup() throws InterruptedException { context.get(CertificateManager.class).startMonitors(); + subscribeToConfigChanges(); super.startup(); }