linuxserver / docker-couchpotato

http://linuxserver.io
GNU General Public License v3.0
154 stars 69 forks source link

Possibility to add option to provide your own Youtube API key #60

Closed clmcgrath closed 2 years ago

clmcgrath commented 2 years ago

The API key used for youtube search is hard coded into the java script , the key is always over its usage cap so the youtube trailers are always broken is it possible to add an environment variable to allow users

the easiest way would probably to do a sed search and replace for the api key on startup using the environment variable value or a default ,

(the file that contains the key is static/scripts/combined.plugins.min.js)

so the file doesnt need to be updated with custom code every time there as CP release

I couldn't find a support link for the developer to submit a bug request , thought this might be a good alternative

{
  "error": {
    "code": 403,
    "message": "The request cannot be completed because you have exceeded your \u003ca href=\"/youtube/v3/getting-started#quota\"\u003equota\u003c/a\u003e.",
    "errors": [
      {
        "message": "The request cannot be completed because you have exceeded your \u003ca href=\"/youtube/v3/getting-started#quota\"\u003equota\u003c/a\u003e.",
        "domain": "youtube.quota",
        "reason": "quotaExceeded"
      }
    ]
  }
}
github-actions[bot] commented 2 years ago

Thanks for opening your first issue here! Be sure to follow the bug or feature issue templates!

aptalca commented 2 years ago

Upstream project is archived and we've been meaning to deprecate this image but we just haven't gotten around to it yet. There won't be any improvements at this point.

We highly recommend switching to radarr instead