Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#42247 closed defect (bug) (invalid)

Imgur oEmbed does not render

Reported by: haroldparis's profile haroldparis Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.8.2
Component: Embeds Keywords:
Focuses: Cc:

Description

Hey there !

Imgur Oembed does not render anymore for galleries, albums or single post.

Example code to add in content :

https://imgur.com/gallery/yJeTl

https://imgur.com/a/ITxk8

https://imgur.com/edZvMqn

Can you replicate this ?

The API answers for those items :

https://api.imgur.com/oembed?url=https://imgur.com/gallery/yJeTl / Ok
https://api.imgur.com/oembed?url=https://imgur.com/a/ITxk8 / Ok
https://api.imgur.com/oembed?url=https://imgur.com/edZvMqn / Sends a 404

I asked Imgur support about this. They told me the calls should be in this form (not sure if it's the way it works right now in Core) :

https://api.imgur.com/oembed?url=/gallery/yJeTl / Ok
https://api.imgur.com/oembed?url=/a/ITxk8 / Ok
https://api.imgur.com/oembed?url=/edZvMqn / Ok

Hope that helps.

Change History (7)

#1 follow-up: @swissspidyCore Committer
7 years ago

Hey there,

Thanks for your report and welcome to Trac! Apparently it's a known problem in the community, see https://community.imgur.com/t/oembed-endpoint/45154/4 for example.

The problems with URLs like https://api.imgur.com/oembed?url=/gallery/yJeTl is that these do not follow the oEmbed specification. The spec says the full URL must be passed via ?url=. That's also what Imgur are advertising in the source code, so the behavior is certainly not correct.

tl;dr: Imgur need to fix this on their end.

#2 in reply to: ↑ 1 @haroldparis
7 years ago

Replying to swissspidy:

tl;dr: Imgur need to fix this on their end.

Will follow up with Imgur API Support for the call format. Thanks !

But, anyway, it should work for my two first examples since API answers correctly. Not sure what's going on right here.

#3 @dd32Lead Developer
7 years ago

The Imgur oEmbed probably isn't working due to #41984

#4 @swissspidyCore Committer
7 years ago

@dd32 I don't see how #41984 should be the reason for that. Imgur serves 404s for some URLs, so they have to change things. Can you perhaps elaborate a bit?

#5 @dd32Lead Developer
7 years ago

@swissspidy I was responding specifically to the previous comment, as to why the first two examples API response works, but doesn't work in WordPress.

Last edited 7 years ago by dd32 (previous) (diff)

#6 @haroldparis
7 years ago

Answer from Imgur API Support :

Hi Harold,
I'm sorry this issue is causing you problems. I'll file a ticket and we will investigate.
Thank you,
Bjorn @ API Support

I'll keep you updated if I've got any news.

#7 @johnbillionCore Committer
7 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Thanks for that @haroldparis. Feel free to re-open this ticket if it turns out something needs fixing or changing in WordPress core.

Note: See TracTickets for help on using tickets.