[wp-trac] [WordPress Trac] #39552: SVG upload support broken in 4.7.1

WordPress Trac noreply at wordpress.org
Thu Jan 19 19:04:41 UTC 2017


#39552: SVG upload support broken in 4.7.1
----------------------------------------+-----------------------
 Reporter:  freakpants                  |       Owner:
     Type:  defect (bug)                |      Status:  reopened
 Priority:  normal                      |   Milestone:  4.7.2
Component:  Upload                      |     Version:  trunk
 Severity:  normal                      |  Resolution:
 Keywords:  needs-unit-tests has-patch  |     Focuses:
----------------------------------------+-----------------------

Comment (by LewisCowles):

 So the fix proposed is to force anything using WordPress to follow the
 mime-type suggested by Fileinfo extension...

 That's potentially undesirable for a few reasons I can think of:

  * It could potentially override the mime type for other file types (wide-
 ranging effects hitting more plugins)
  * It ties WP to a specific implementation and update path (other PHP exts
 have died in the past; we maybe shouldn't assume Fileinfo will always be
 there or rely on it too much)
  * Just by avoiding the 'image/' prefix we should be able to avoid this
 altogether

 IMO if anything removing superficial checks, giving advice to use a
 dedicated program would make it easier to maintain and limit how much WP
 has to do and maintain.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/39552#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list