WebP Upload Issue with undefined height about payload

[todor0v]

I have the exact same issue on Payload 3.0.0-beta.43 and Sharp 0.33.4.

If I do not specify the height at all, I get:
ERROR: FileUploadError: There was a problem while uploading the file. at generateFileData (webpack-internal:///(rsc)/./node_modules/payload/dist/uploads/generateFileData.js:232:15)

If I specify the height as any number or undefined, I get:
ERROR: ValidationError: media validation failed: sizes.image.height: Cast to Number failed for value "NaN" (type number) at path "height" at Document.invalidate (/node_modules/mongoose/lib/document.js:3177:32)

For me the suggested maxOffsetY fix did not work.

[kirillminiaev]

@kirillminiaev Of course:

@todor0v your error seems to be related more to your database: MongoDB. It's as if the database model expects there to be a "height" value for your resizing and when a NaN value is passed, the internal Model validation kicks in and spews the error. So in your case, I would think either a height is required (and the Payload team needs to address a workaround) or code needs to ensure that a NaN is not passed to the model. Ultimately, it could be that whatever data is passed to the Next create.js:20:17 line needs to be pre-processed to validate the height parameter. At least that is where I would start my debugging process if I were you.

[denolfe]

Fairly certain this is the fix #6729

Let's verify in the next 2.x release.

[kirillminiaev]

I have the exact same issue on Payload 3.0.0-beta.43 and Sharp 0.33.4.

If I do not specify the height at all, I get: ERROR: FileUploadError: There was a problem while uploading the file. at generateFileData (webpack-internal:///(rsc)/./node_modules/payload/dist/uploads/generateFileData.js:232:15)

If I specify the height as any number or undefined, I get: ERROR: ValidationError: media validation failed: sizes.image.height: Cast to Number failed for value "NaN" (type number) at path "height"

For me the suggested maxOffsetY fix did not work.

@todor0v can you post a full error stack-trace. Seems you may need to adjust the code pertaining to the specific Cast to Number failed for value "NaN" (type number) at path "height" error for a fix for your version of Payload.

[todor0v]

@kirillminiaev Of course:

[kirillminiaev]

Fairly certain this is the fix #6729

Let's verify in the next 2.x release.

@denolfe thank you. Looking forward to the next release with the fix implemented.

[kirillminiaev]

@PatrikKozak, just upgraded to 2.20.0 and the issues persists. Not sure if you were planning to push the fix with this next release. My original fix patched the issue in 2.20.0

[PatrikKozak]

Hey @kirillminiaev

I have a fix for the issue here.

V3 fix is here

We'll get these merged either today or tomorrow!

[kirillminiaev]

Thank you @PatrikKozak!

[github-actions]

This issue has been automatically locked.
Please open a new issue if this issue persists with any additional detail.