Commitaafade6

committed

minor#20893 [Validator] Highlightextensions instead ofmimeTypes forFile constraint (benito103e)

This PR was submitted for the 7.2 branch but it was merged into the 6.4 branch instead.Discussion----------[Validator] Highlight `extensions` instead of `mimeTypes` for `File` constraintYou should always use the extensions option instead of mimeTypes except if you explicitly don't want to check that the extension of the file is consistent with its content (this can be a security issue).By default, the extensions option also checks the media type of the file.Commits-------3474a09 Highligh 'extensions' instead of 'mimeTypes' for File constraint

2 parents042f70d +3474a09 commitaafade6Copy full SHA for aafade6

File tree

1 file changed

-5

lines changed

controller
- upload_file.rst

1 file changed

-5

lines changed

`‎controller/upload_file.rst`

Lines changed: 2 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -77,11 +77,8 @@ so Symfony doesn't try to get/set its value from the related entity::`
`77`	`77`	`'constraints' => [`
`78`	`78`	`new File([`
`79`	`79`	`'maxSize' => '1024k',`
`80`		`- 'mimeTypes' => [`
`81`		`- 'application/pdf',`
`82`		`- 'application/x-pdf',`
`83`		`- ],`
`84`		`- 'mimeTypesMessage' => 'Please upload a valid PDF document',`
	`80`	`+ 'extensions' => ['pdf'],`
	`81`	`+ 'extensionsMessage' => 'Please upload a valid PDF document',`
`85`	`82`	`])`
`86`	`83`	`],`
`87`	`84`	`])`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commitaafade6

File tree

1 file changed

1 file changed

`‎controller/upload_file.rst`

0 commit comments