search

soulim / oembed

A slim library to work with oEmbed format.
http://sul.im/oembed
MIT License
65 stars 7 forks source link

Failure on second redirect #6

Closed rossjourdain closed 8 years ago

rossjourdain commented 8 years ago

This is a bit of an edge case that struck today. If you request an oembed URL that results in a redirect, that results in a redirect, you get an error.

It happens on 'fetch' because http.rb:21, thinks it has an options hash but it has been passed a fixnum by http.rb:39.

Hope that makes sense!

soulim commented 8 years ago

Seems like the bug was there for ages. Thanks a lot @rossjourdain! 😄👍

rossjourdain commented 8 years ago

No worries - bit of an edge case... Thanks for the gem @soulim