igniterealtime / openfire-hazelcast-plugin

Adds support for running multiple redundant Openfire servers together in a cluster
10 stars 12 forks source link

Fix NullPointerException logging the NodeId #52

Closed seltihood closed 3 years ago

seltihood commented 3 years ago

Hi, We were having some exceptions because route was null (Line 77: NodeID nodeID = route.getNodeID();) Could you please merge this fix? Thanks!

seltihood commented 3 years ago

Hi, I put both commits in different pull requests in case you were not interested on the other one :)

seltihood commented 3 years ago

@GregDThomas Could you merge this one too? We need this because we are having NullPointers exceptions due to the route being null when getting the variable to log it in line 77 (https://github.com/igniterealtime/openfire-hazelcast-plugin/pull/52/files#diff-0ddf0d547dd3a57d1b7933fda1715a96L77)