I have installed the nodejs module and configured it. I have tried to test it using broadcast message but its not sending messages.

CommentFileSizeAuthor
#2 nodejs_notify_not-2834552-2.patch502 bytessriharsha.uppuluri
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sriharsha.uppuluri created an issue. See original summary.

sriharsha.uppuluri’s picture

sriharsha.uppuluri CreditAttribution: sriharsha.uppuluri at Azri Solutions commented
FileSize
nodejs_notify_not-2834552-2.patch502 bytes

After spending a 3 hours of time I have found that the safeNodeServerVerion() function is doing a nodejs version check and based on that its displaying the messages. I think there is a type casting issue here. Attaching the patch for the same.

glekli’s picture

glekli CreditAttribution: glekli commented

The version number contains periods, which means it's not a valid int. Casting to int would cause data loss.
By the way, this type of check was refined in the d7 version, and needs to be ported to d8. I'll look into that soon.