[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