ExpressAdapter exposes the wrong Http Server

[BrunnerLivio]

I'm submitting a...

[ ] Regression 
[ ] Bug report
[ ] Feature request
[x] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

When using Nest with the Express adapter it returns the wrong Http Server when calling HttpServer.getHttpServer().

Example with ExpressAdapter

// main.ts
const app = await NestFactory.create(AppModule);
....
// app.service.ts
@Injectable()
export class AppService {
  constructor(
    @Inject(HTTP_SERVER_REF)
  private server) {
    // Prints "EventEmitter" --> Wrong
    console.log(this.server.getHttpServer().constructor.name);
...

Example with FastifyAdapter

// main.ts
const app = await NestFactory.create(AppModule, new FastifyAdapter());
....
// app.service.ts
@Injectable()
export class AppService {
  constructor(
    @Inject(HTTP_SERVER_REF)
  private server) {
    // Prints "Server" (instance of the http package) --> Good
    console.log(this.server.getHttpServer().constructor.name);
...

Expected behavior

getHttpServer should expose the server instance of the http npm package independent of the adapter.

What is the motivation / use case for changing the behavior?

The http.server instance is needed for the terminus implementation --> See #966

[kamilmysliwiec]

It should be fixed in the v5.3.0

[BrunnerLivio]

@kamilmysliwiec Just checked and it works like a charm. Thanks a lot for the quick fix!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.