callback is not a function about nestjs-rmq

[i-kitaev]

Thanks your :)

[AlariCode]

@vnkitaev, thanks for report, I'll check with latest version of NestJS.

[AlariCode]

@vnkitaev, tested with:

"dependencies": {
    "@nestjs/common": "^5.3.6",
    "@nestjs/core": "^5.3.6",
    "@nestjs/microservices": "^5.3.6",
    "@nestjs/testing": "^5.0.0",
    "@nestjs/websockets": "^5.0.0",
    "nestjs-rmq": "0.1.1"
  },

No error. Everything seams fine.

Could you provide actual code, that causes this error?

[i-kitaev]

await NestFactory.createMicroservice(ApplicationModule, {
      strategy: new ServerRMQ({
        urls: ['amqp://guest:guest@localhost:5672'],
        queue: 'test',
        queueOptions: { durable: false }
      })
    });

await this.app.listen(3000);

ApplicationModule is empty.

If the callback is optional, maybe add check is callback a function?

nestjs-rmq/lib/server/server.ts

Line 41 in 8b576be

private async start(callback?: () => void) {

[AlariCode]

@vnkitaev, I updated package to 0.1.2 and added additional check if callback is a function. Now your error must be gone. But NestJS insists, that callback must be a function. Even if I set it to number, listen() still warn about error:

When you start microservice you don't specify any port: https://docs.nestjs.com/microservices/basics

import { NestFactory } from '@nestjs/core';
import { Transport } from '@nestjs/microservices';
import { ApplicationModule } from './app.module';

async function bootstrap() {
  const app = await NestFactory.createMicroservice(ApplicationModule, {
    transport: Transport.TCP,
  });
  app.listen(() => console.log('Microservice is listening'));
}
bootstrap();

But additional check will be ok) Thanks for submitting an issue 👍