[wp-trac] [WordPress Trac] #17106: oEmbed - PollDaddy regex needs updating
WordPress Trac
wp-trac at lists.automattic.com
Mon Apr 11 17:32:26 UTC 2011
#17106: oEmbed - PollDaddy regex needs updating
--------------------------+------------------------------
Reporter: r-a-y | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Embeds | Version: 3.1
Severity: normal | Resolution:
Keywords: has-patch |
--------------------------+------------------------------
Comment (by duck_):
I think we will want to add an escaped dot to the regex:
{{{
#http://(.+)?\.polldaddy\.com/.*#i
}}}
to avoid matching completely different domains, e.g.
http://notpolldaddy.com/, and only allow polldaddy.com subdomains.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/17106#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list