fix npe when no public address for MarathonTaskLocation.targetMachine

brooklyncentral / clocker

Apache Brooklyn cloud native infrastructure blueprints

Apache License 2.0

426 stars 66 forks source link

Closed johnmccabe closed 8 years ago

johnmccabe commented 8 years ago

Fixes the issue raised in #264, have tested in Blue Box VPN in both a publically accessible cluster and one accessible only via VPN.

@grkvlt I don't have a solid grasp on the handling of hostnames in this case so suspect this may be too simplistic a fix.

grkvlt commented 8 years ago

@johnmccabe @aledsage Are you sure this is right?