#20322 closed defect (bug) (fixed)

viddler embed not working

Reported by: arena Owned by: nacin
Priority: normal Milestone: 3.4
Component: Embeds Version: 3.4
Severity: normal Keywords: has-patch
Cc:

Description

when having this in a post :

http://www.viddler.com/explore/BigAppleChannel/videos/113/

should display the video !

but nothing appears

Attachments (2)

wp-oembed-data2html-sanitition.diff (1.9 KB) - added by nacin 14 months ago.
Patch from mdawaffe.
20322.diff (1.7 KB) - added by nacin 14 months ago.
Smooth the edges of the initial patch. Always use the filter.

Download all attachments as: .zip

Change History (12)

Doesn't work on my 3.3 install, but it works on my wordpress.com blog.

I get this error on 3.3

Warning: strpos() expects parameter 1 to be string, object given in /home/foobar/public_html/wp-includes/class-oembed.php on line 268

Catchable fatal error: Object of class stdClass could not be converted to string in /home/foobar/public_html/wp-includes/class-oembed.php on line 269

If I use [viddler id=8918af90]

I get [ERROR: An invalid Viddler shortcode format was used. Please check your code.]

I can reproduce it with Viddler's sample video: http://www.viddler.com/explore/cdevroe/videos/424/ (just to make sure it wasn't the vid itself)

Version 0, edited 14 months ago by Ipstenu (next)

Contrary to their docs, Viddler seems to actually return an object for the html value now, which is not in line with the oEmbed spec. Example

We've been discussing this internally on the security team list and had planned to make it public. Someone reached out to Viddler but apparently they have not acted on this.

Let's add some poka-yoke here to ensure we are always dealing with a scalar value. And then get Viddler to fix their broken endpoint.

Last edited 14 months ago by nacin (previous) (diff)

nacin14 months ago

Patch from mdawaffe.

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to Future Release
  • Milestone changed from Future Release to 3.4

nacin14 months ago

Smooth the edges of the initial patch. Always use the filter.

  • Severity changed from major to normal
  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [20539]:

Add sanity checks to WP_oEmbed::data2html() to ensure we are working with scalar values. Always use the filter. props mdawaffe, fixes #20322.

Note: See TracTickets for help on using tickets.