fix: convert msg to array msg

[wwayne]

The method deferPublish in nsq.js doesn't covert msg into msg array code in nsq.js, so sending string msg with delay parameter will get an error from NSQ.

We can force converting msg into array to avoid the bug from nsq.js

[CCharlieLi]

I'm not sure we should convert this implicitly as we are following the same format as NSQ.js https://github.com/dudleycarr/nsqjs#a-writer-example, even though I'm not sure about the intention of sending nsq defer message with a array of string.

Regarding that this is a breaking change, we need to consider potential risk and edge cases of converting message into array. @wwayne

[wwayne]

I think that is a bug from nsq.js, he forgot converting msg to array msg for deferPublish and missed the validation in the unit test, I sent him a PR as well. nsq.js publish and deferPublish method

NSQ itself accepts message array only or it will return an error. So for the nsq-strategy, the example in the README for delay msg actually will get an error

 p.connect((errors) => {
    p.produce('topic', 'message', { retry: {
      retries: 3,
      minTimeout: 300
    }, delay: 2000 }, (err) => {
      if (err) {
        console.log(err);
      }
    });
 });

[CCharlieLi]

FYI, I also found related issue dudleycarr/nsqjs#274

[wwayne]

and I found his PR after I sent mine, for the same issue 🤣
the author seems inactive in the maintaining

[xavierchow]

closing, we only need to use nsqjs 0.13.0

hmm, figured out dudleycarr/nsqjs#275 is not merged.