Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requested page error: 500 on a Facebook video #106

Closed
j0k3r opened this issue Feb 16, 2016 · 4 comments
Closed

Requested page error: 500 on a Facebook video #106

j0k3r opened this issue Feb 16, 2016 · 4 comments

Comments

@j0k3r
Copy link
Contributor

j0k3r commented Feb 16, 2016

We got this error while trying to embed a Facebook video (I tried one of urls from the tests array in facebook.video.js):

http://medium.iframe.ly/api/oembed?url=https://www.facebook.com/joe.yu.94/videos/10206321173378788

{
    "status": 408,
    "error": "Requested page error: 500"
}

Also, here is the stacktrace from one of our server:

==> logs/node.err <==
Error: Requested page error: 500
    at /space/www/iframely/data/releases/20151215135529/modules/api/views.js:88:29
    at /space/www/iframely/data/releases/20151215135529/node_modules/async/lib/async.js:52:16
    at /space/www/iframely/data/releases/20151215135529/node_modules/async/lib/async.js:1166:30
    at asyncMethodCb (/space/www/iframely/data/releases/20151215135529/lib/core.js:1290:28)
    at /space/www/iframely/data/releases/20151215135529/lib/core.js:269:29
    at process._tickCallback (node.js:355:11)

Also, we are using 0.9.1 since (I guess) you forgot to tag 0.9.2.

iparamonau added a commit that referenced this issue Feb 16, 2016
@iparamonau
Copy link
Member

Thanks for reporting. It's the same issue as #102 - FB sporadically returns 500 errors when you curl them. It looks the occurrence became more often now though.

The FB parser has been switched to oEmbed-only now whenever possible. That should help avoid curling FB (and so getting errors).

Pull it from develop branch, or wait a bit until we run a complete test and release v0.9.3

@j0k3r
Copy link
Contributor Author

j0k3r commented Feb 16, 2016

I'll wait for 0.9.3, thanks for the quick fix !

@iparamonau
Copy link
Member

@j0k3r I posted a question to you last week about medium plugin btw. Let's try to figure that out too.

@j0k3r
Copy link
Contributor Author

j0k3r commented Feb 16, 2016

Yep, we need to think about that.

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

No branches or pull requests

2 participants