Skip to content
This repository was archived by the owner on Jan 13, 2022. It is now read-only.

adds absolute path to oembed GET as DEFAULT_ENDPOINT is not supported #163

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

adds absolute path to oembed GET as DEFAULT_ENDPOINT is not supported #163

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e96a951
adds absolute path to oembed GET as DEFAULT_ENDPOINT is not supported
JONBRWN May 7, 2015
5a3fd71
updates oembed to use forward slash as opposed to absolute path
JONBRWN May 8, 2015
beda2f4
instagram updated oembed path to redirect to publicapi/oembed
JONBRWN Jun 18, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion lib/instagram/client/embedding.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ module Embedding
def oembed(*args)
url = args.first
return nil unless url
get("oembed?url=#{url}", {}, false, false, true)
get("/https://api.instagram.com/oembed?url=#{url}", {}, false, false, true)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can just change this to:

get("/oembed?url=#{url}", {}, false, false, true)

That's how we've monkey-patched it to fix it. That way, it still uses the configuration's endpoint, but throws away the /v1 path.

end
end
end
Expand Down