Catch if Rabbit server is down about nestjs-rmq

[ebadia]

@AlariCode Sorry, my fault... I did not upgrade the package to incude the changes... Now the error message has disapeared. Thanks!

[AlariCode]

@ebadia, thanks for report! I'll look into this issue. I think I'll need to add additional check in package.

[ebadia]

Thanks for making this work so well with nestjs! Just as an idea, I saw that in connect you only return the resolve case, perhaps rejecting the promise could help.

[AlariCode]

@ebadia, I updated package to 0.1.1 with better connection management system.

Now you can catch error on client:

try {
       return this.client.send<string, string>({ cmd: 'test' }, msg);
} catch (err) {
      console.log('Error:' + err);
 }

And server and client will reconnect when RMQ server will be available (if it was down):

Furthermore, I added support for clustering. You can specify more then one url on client or server:

client = new ClientRMQ({
    urls: [`amqp://login:password@host`],
    queue: 'test',
    queueOptions: { durable: false }
}); 

Please note that now you need to use urls option instead of url and pass string[] instead of string.

[ebadia]

That's great, I'll give it a try. Thanks.

[ebadia]

@AlariCode this solves the error problem when sending the message. And for sure it's great. But the system still returns an error message when I try to do the new ClientRMQ

(node:54282) UnhandledPromiseRejectionWarning: Error: connect ECONNREFUSED 127.0.0.1:5672

It seems that we have an unhandled promise rejection there too. This does not stops the app and the try/catch you have added helps a lot. But perhaps this "warning" should be considered for the future.

[AlariCode]

@ebadia this is strange. It connects only when first message is sent. This code:

const client = new ClientRMQ({
            urls: [`amqp://${config.default.localhost.amqp.login}:${config.default.localhost.amqp.password}@${config.default.localhost.amqp.host}`],
            queue: 'test',
            queueOptions: { durable: false },
        });

doesn't cause client to connect. And this one does:

this.client.send<string, string>({ cmd: 'test' }, msg);

Could you please show you code with error? This example catches an error perfectly:

@Controller()
export class AppController {
    client = new ClientRMQ({
        urls: [`amqp://${config.default.localhost.amqp.login}:${config.default.localhost.amqp.password}@${config.default.localhost.amqp.host}`],
        queue: 'test',
        queueOptions: { durable: false },
    });

    @Get('a')
    a(): Observable<string> {
        try {
            return this.client.send<string, string>({ cmd: 'test' }, 'test');
        } catch (err) {
            console.log('Got error: ' + err);
        }
    }
}

[AlariCode]

Thanks for bug report! I’m closing this issue.