-
Notifications
You must be signed in to change notification settings - Fork 37
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
[BUG] Adding YouTube URL not working #39
Comments
This comment was marked as outdated.
This comment was marked as outdated.
Nevermind, I found out the issue. The API key used in this project exceeded request quota. I get the following response if I do the request with the default api key:
I think the best solution should be to allow users to use their own api key, maybe by adding a textbox in the preferences. what do you think? |
I'm working on a pull request for this. Did you ever end up looking into this further?
Edit: Did more research, did not realize OAuth and the API Key used the same quota. I can absolutely add a Key field to options, including the Client ID and Secret for OAuth (not sure if having the same client and a different key would set off Google security). |
@cloewen8 I temporary solved the issue by manually generating an api key on my account and changing it in the dll. This is the value that I replaced in mine, without touching client_id or client_secret: |
In testing I noticed some calls would not work unless the OAuth and key where from the same projects. In those cases I'd have to use the build-in key (I'm providing an option to supply a custom client). |
I recompiled from latest master, but I haven't checked it, I don't use AIMP anymore |
Working perfectly now with my own developer keys. thanks! |
Thanks, it works fine with v1.8. You can close the issue once this version is officially released. |
Comrades, so what about the possibility to add keys to the settings tab of the plugin? As I understand it would solve all the problems. Now you can't use the plugin if you don't know how to build a project from sources. |
Now you can't read properly what @AdrianEddy said above, apparently 🤏:
|
But there is no x64 build in this archive and it is not in the releases section… Ok, I reinstalled AIMP to 32bit and yes, the 1.8 version workes with key property. Thank you. |
What is needed to get a x64 build working? I'd be interested in contributing. I no longer use this plugin, but feel it is very important in the Aimp ecosystem. |
x64 build is available here #30 (comment) |
Add YouTube URL
andCheck for new videos
not working anymore since September 2023.I checked the config file and found out that the url saved to retrieve video/playlist details returns a 403 error. Maybe that could be the issue.
The text was updated successfully, but these errors were encountered: