search

kuujo / vertigo

Flow-based programming for the Vert.x application platform.
Apache License 2.0
155 stars 24 forks source link

NPE in abstract coordinator #29

Closed kuujo closed 10 years ago

kuujo commented 10 years ago

The AbstractCoordinator is getting a bad message from somewhere that is causing an NPE. See https://groups.google.com/forum/#!topic/vertx-vertigo/24VDLdNu8Sc. This should be guarded against since Vertigo shares communication with the rest of Vert.x and there's a lot of potential for bad messages to get into the coordination process.

kuujo commented 10 years ago

This was caused by Vertigo internally when switching over methods of coordination. The components were sending commands to the coordinator that the coordinator didn't understand. This should be fixed

22be523267b2be232deb91b7a750aab8742c880f

Also, the coordinator should check for null actions in general in order to prevent messages from other Vert.x systems from interfering with Vertigo coordination.