RabbitMQ messages requeued by default in case of error, in contradiction with the docs

[rthouvenin]

The README of the rabbitmq package says that the default error handling behaviour is to nack messages without requeue:

nestjs/packages/rabbitmq/README.md

Line 616 in 96994a3

The default is `defaultNackErrorHandler` and it just nack the message without requeue (which is usually ok to avoid the message coming back in the queue again and again)

However the behavior that I observed in my app is that messages are re-queued, causing an infinite loop. And I think if I understand the code correctly, this is indeed what's currently coded, as the base default config has defaultSubscribeErrorBehavior = MessageHandlerErrorBehavior.REQUEUE:

nestjs/packages/rabbitmq/src/amqp/connection.ts

Line 109 in 96994a3

defaultSubscribeErrorBehavior: MessageHandlerErrorBehavior.REQUEUE,

This discrepancy is confusing. I'm happy to submit a PR, but not sure which is the desired outcome: fix the documentation or change the code to match the documentation?