-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ソースコードを読んでみる #1
Comments
NestJSで問題が発生しているのはこの部分 https://github.com/nestjs/nest/blob/master/packages/common/pipes/validation.pipe.ts#L124 let entity = classTransformer.plainToClass(
metatype,
value,
this.transformOptions,
); どうやら |
詳しく追って改修しようとしたところ同様の問題に対するPRが出ていた。 |
ValidatePipeをオーバーライドするか |
I decided to set |
既存箇所に影響が出るため根本解決にならず |
移行先を探そう。 |
forkしたライブラリを作ることで解決 |
わかってる情報
multipart/form-data
を使用するとstring
として情報が渡される(application/jsonなどはjsonのパーサーが使用されると思われ)
enableImplicitConversion: true
を渡すと暗黙的な変換が行われる為、この時点で文字列'false'
がbooleantrue
に変換される。enableImplicitConversion: true
になってると@Transform(({ value }) => value === 'true')
を書いても意味がないThe text was updated successfully, but these errors were encountered: