Skip to content
This repository has been archived by the owner on Dec 26, 2019. It is now read-only.

Failure on second redirect #6

Merged
merged 5 commits into from
Jul 1, 2016
Merged

Failure on second redirect #6

merged 5 commits into from
Jul 1, 2016

Conversation

rossjourdain
Copy link
Contributor

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
Copy link
Owner

soulim commented Jul 1, 2016

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

@soulim soulim merged commit ff28a20 into soulim:master Jul 1, 2016
@rossjourdain
Copy link
Contributor Author

rossjourdain commented Jul 3, 2016

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants