ValidationPipe Date query parameter | NestJS | Page 1

novel cloud Dec 18, 2023, 4:08 PM

#

Hello, I need to ask regarding change in ValidationPipe in versions after 10.2.7. This is the commit I found causing my issue:
https://github.com/nestjs/nest/commit/f3141fbfdabd66de4485d76ce22c6e9b16dd55f8

Before this change, I could have @Query('myparam') Date date, and the Date would get transformed to Date object. However after this commit, its it not being deserialized and I was not able to find it in changelog of the newer versions. Can someone explain this to me, if current behavior is intended (and what should I change), or if it is a bug?

GitHub

fix: add date to ignored metatypes · nestjs/nest@f3141fb

mystic nymph Dec 18, 2023, 4:17 PM

#

This was an intended change. We don't normally want to deserialize built in classes. If that's the desired functionality, you should create a DTO that specifies this

novel cloud Dec 18, 2023, 4:19 PM

#

Sure, I'll update that. Might this be good think to put in changelog? Searching nestjs implementation when tests started to fail was not really convenient 😄

mystic nymph Dec 18, 2023, 4:19 PM

#

Dunno why Kamil didn't put it in a changelog to be honest

#ValidationPipe Date query parameter