search

MeowWolf / node-red-contrib-amqp

AMQP nodes for node-red
22 stars 18 forks source link

'requeue' parameter should be default True #39

Closed pauldeng closed 3 years ago

pauldeng commented 3 years ago

Hi @amodelbello,

The Minor Issue

Ver: 2.3.0

I am testing the newly added nack feature. I found the default value of 'requeue' parameter should be TRUE.

The 'requeue' parameter is also used in nackAll and reject.

However, the default value is FALSE in the code.

The Workaround Explicitly set msg.manualAck.requeue to TRUE.

Cheers, Paul