[wp-trac] [WordPress Trac] #32359: Add oEmbed support for WordPress.com

WordPress Trac noreply at wordpress.org
Thu Jun 4 21:25:32 UTC 2015


#32359: Add oEmbed support for WordPress.com
-------------------------------------+------------------
 Reporter:  johnbillion              |       Owner:
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  4.3
Component:  Embeds                   |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  needs-patch 2nd-opinion  |     Focuses:
-------------------------------------+------------------

Comment (by georgestephanis):

 I'd object to that regex, it feels overly broad when specifying the
 subdomains.

 `#https?://[^/]+\.wordpress\.com/[\d/]+/.+#i`

 I'd far sooner see the subdomain logic done as `[a-z\d]{4,50}` --
 alphanumeric, and between four and fifty characters, inclusive.  WPCOM
 subdomains also cannot contain dashes or underscores, so that regex should
 be much more accurate, unless you intended to also include things like
 `en.blog.wordpress.com` or the like.

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


More information about the wp-trac mailing list