No validation for default image user can upload any file type.

CommentFileSizeAuthor
#1 2322377_1_TEST_ONLY.patch852 bytesslashrsm
#1 2322377_1.patch1.67 KBslashrsm
Screenshot from 2014-08-16 12:18:38.png29.09 KBroaldumandal
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

slashrsm’s picture

slashrsm CreditAttribution: slashrsm commented
Version: 8.0.0-alpha14 » 8.0.x-dev
Status: Needs work » Needs review
Issue tags: +Media Initiative
FileSize
2322377_1.patch1.67 KB
2322377_1_TEST_ONLY.patch852 bytes

Fix + test coverage.

Status: Needs review » Needs work

The last submitted patch, 1: 2322377_1_TEST_ONLY.patch, failed testing.

slashrsm’s picture

slashrsm CreditAttribution: slashrsm commented
Status: Needs work » Needs review

Patch with FIX passed.

marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Needs review » Reviewed & tested by the community

Looks good

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed c281ef5 and pushed to 8.0.x. Thanks!

  • alexpott committed c281ef5 on 8.0.x 
    Issue #2322377 by slashrsm | roald.umandal: Fixed No validation for...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Issue tags: -Media Initiative +D8Media

Fix media tag.