[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