tree 14305e3bb6f458470ffad48ef614f8ab454d7d4a
parent 0176130ee6f7795de0ad42f6c9990fddae43e067
author Quan Tran <hqtran@linagora.com> 1713521886 +0700
committer Rene Cordier <rene.cordier@gmail.com> 1713751842 +0700

[FIX] GroupRegistrationHandler's scheduler should be final

And be initialized in the constructor.

Given the RabbitMQEventBus beans are limited and the group use case is soon or late being used, we do not need lazy initialization for the scheduler IMO.

Otherwise, James could restart the `GroupRegistrationHandler` while it is not started yet (e.g. a RabbitMQ node is down before any group listener is registered) which leads to `java.lang.NullPointerException: scheduler` and fail the `RabbitEventBusConsumerHealthCheck`.