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

YouTube now returning broken video #3

Closed
stephenyeargin opened this issue May 12, 2015 · 5 comments · Fixed by #4
Closed

YouTube now returning broken video #3

stephenyeargin opened this issue May 12, 2015 · 5 comments · Fixed by #4
Labels

Comments

@stephenyeargin
Copy link
Member

Reproduction steps:

user> hubot yt star wars
hubot> http://www.youtube.com/watch?v=UKY3scPIMd8&feature=youtube_gdata

This is a result of an announced change in their API documentation. The fix will be to switch to an authenticated mode (via OAuth2) to restore functionality.


New endpoint example:

GET https://www.googleapis.com/youtube/v3/search?part=snippet&q=star+wars&key={YOUR_API_KEY}
@stephenyeargin
Copy link
Member Author

This is actually a lot easier than I thought it was. You can simply go into the Google API Console with an authenticated account and obtain a non-auth'd token rather than going through the entire OAuth2 song and dance. As this package does not interact with user data, this is the ideal route.

Public API access
Use of this key does not require any user action or consent, does not grant access to any account information, and is not used for authorization.

screen_shot_2015-05-12_at_6_08_02_pm

@seanCodes
Copy link

+1, love to see the PR merged and the issue fixed. 😁

@stephenyeargin
Copy link
Member Author

@seanCodes #4 should take care of it. Can you give it a spin? Should be a simple matter of updating your package.json file to include:

{
    "hubot-youtube": "git://github.com/stephenyeargin/hubot-youtube.git#bug/3-broken-youtube-search"
}

If it passes, adding a 👍 to the PR may help move it along.

@SarahRiley
Copy link

@stephenyeargin I tried out your PR - works nicely, thank you!

@ndaversa
Copy link

ndaversa commented Jun 3, 2015

I also ran this fork locally and it worked as reported 👍

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

Successfully merging a pull request may close this issue.

5 participants