Nest throws an internal server error despite specifying ConflictException | NestJS | Page 1

mint wren Jul 25, 2023, 2:00 AM

#

is this a nestjs v10 change?

#

 @Post(":multiTradeId/acknowledge")
  @HttpCode(HttpStatus.OK)
  async acknowledgeMultiTrade(@Param("multiTradeId") multiTradeId: string) {
    throw new BadRequestException("Not implemented.");
    await this.multiTradeService.acknowledge(multiTradeId);
  }

#

{
  "statusCode": 500,
  "message": "Internal server error"
}

hollow vigil Jul 25, 2023, 2:01 AM

#

Everything is udated to v10?

mint wren Jul 25, 2023, 2:03 AM

#

idk why i said that in the project im in its v9

hollow vigil Jul 25, 2023, 2:04 AM

#

Is everything on the same nest version?

mint wren Jul 25, 2023, 2:06 AM

#

i believe so

hollow vigil Jul 25, 2023, 2:06 AM

#

npx nest info?

coarse fieldBOT Jul 25, 2023, 2:06 AM

#

Please run the command npx -y @nestjs/cli info and paste the output in a code block. This will help us determine if there is a version issue in your packages and which version of nest we are triaging.

hollow vigil Jul 25, 2023, 2:06 AM

#

What's the stack trace in the terminal?

mint wren Jul 25, 2023, 2:15 AM

#

i assumed it was a package version different from another so i made sure all versions were the same

#

(it sa monorepo)

#

for each sub app

hollow vigil Jul 25, 2023, 2:15 AM

#

Oh boy. what kind of monorepo?

mint wren Jul 25, 2023, 2:15 AM

#

nothing big just two small sub apps and i have core and common installed at root

#

turborepo

#

this is the more interesting error

#

hollow vigil Jul 25, 2023, 2:16 AM

#

Do you have a common node_modules or a node_modules per app?

mint wren Jul 25, 2023, 2:16 AM

#

yea using pnpm so they all link back to the same place

hollow vigil Jul 25, 2023, 2:16 AM

#

mint wren yea using pnpm so they all link back to the same place

Don't care that they link back. Does each app have a node_modules with @nestjs/common or @nestjs/core in it?

mint wren Jul 25, 2023, 2:17 AM

#

yes

#

no

hollow vigil Jul 25, 2023, 2:17 AM

#

mint wren yes

Okay, that's an issue due to how JS realms work

hollow vigil Jul 25, 2023, 2:17 AM

#

mint wren no

Well, which is it?

mint wren Jul 25, 2023, 2:17 AM

#

they don't

#

they both use platform-fastify and each have their own if that helps

hollow vigil Jul 25, 2023, 2:18 AM

#

That might be an issue

#

Because @nestjs/platform-fastify has peer deps on @nestjs/common

mint wren Jul 25, 2023, 2:18 AM

#

does that explain the type error im getting for FastifyAdapter

hollow vigil Jul 25, 2023, 2:19 AM

#

There's a decent chance, yeah

mint wren Jul 25, 2023, 2:19 AM

#

i'll try installing it at root then

hollow vigil Jul 25, 2023, 2:19 AM

#

Especially if it doesn't match the common and core version exactly

mint wren Jul 25, 2023, 2:25 AM

#

after i tried making sure all packages were the same version i started getting this error (instead of the app running and not handling the thrown error properly)

Nest can't resolve dependencies of the EventSubscribersLoader (DiscoveryService, EventEmitter, EventsMetadataAccessor, MetadataScanner, ?). Please make sure that the argument ModuleRef at index [4] is available in the EventEmitterModule context.

I remember this error and its usually when you have a version mismatch, I just fixed that by pinning versions (9.4.0 vs ^9.4.0). So now I'm back to issue with the http error not being handled properly

#

does this clear up it being a version mismatch?

hollow vigil Jul 25, 2023, 2:27 AM

#

Yeah, if ModuleRef or Reflector ever show up, it's usually a mutliple version install issue or version mismatch

mint wren Jul 25, 2023, 2:30 AM

#

well back to square one

#

ok

#

i got it was still a version mismatch

#

but with rxjs

#Nest throws an internal server error despite specifying ConflictException