-
Notifications
You must be signed in to change notification settings - Fork 6
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
ID not found for vain... #65
Comments
Checking for 404 should be sufficient |
Thanks for stopping by.
You are welcome to submit a PR to fix this. Thanks!
…On Thu, Jan 19, 2017 at 10:41 AM, Oded Shafran ***@***.***> wrote:
Checking for 404 should be sufficient
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#65 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHEL02Kf-tYaJWhdU5zNBq6mhXmRHvouks5rT65ygaJpZM4LodKA>
.
--
Claudio Baccigalupo | Manager, Software Engineering | Fullscreen, Inc.
<http://www.fullscreen.com>
|
@himberjack The other PR was merged but this issue is still relevant. In short, some videos don't have a What's a different way that we can use to check whether a Facebook URL is valid and matches a post? |
It is a part of my PR. please see it. 404 should be fine. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ID 2003562643204142 for instance claims to be not found. However after looking at the HTML, it doesnt have meta description tag as the code checks... maybe the validity of the existence of an asset should be checked other way.
The text was updated successfully, but these errors were encountered: