Skip to content

Change default defParamCharset from latin1 to utf-8#1368

Open
UlisesGascon wants to merge 1 commit intomainfrom
major/default-utf8
Open

Change default defParamCharset from latin1 to utf-8#1368
UlisesGascon wants to merge 1 commit intomainfrom
major/default-utf8

Conversation

@UlisesGascon
Copy link
Member

BREAKING CHANGE: The default character set for parsing multipart form-data headers (e.g., filenames) is now 'utf-8' instead of 'latin1'.

This aligns with modern web standards where UTF-8 is ubiquitous. Users who need the previous behavior can explicitly set: multer({ defParamCharset: 'latin1' })

Continuation of #1210

BREAKING CHANGE: The default character set for parsing multipart form-data headers (e.g., filenames) is now 'utf-8' instead of 'latin1'.

This aligns with modern web standards where UTF-8 is ubiquitous. Users who need the previous behavior can explicitly set: `multer({ defParamCharset: 'latin1' })`
Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already had the v3 branch, right? Those changes should go there. But more importantly, I don’t think that option is even in v3 yet, which would be worth double-checking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants