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

Adding Google Cloud API key for YouTube causes crash #121

Closed
Nezteb opened this issue Dec 28, 2023 · 2 comments
Closed

Adding Google Cloud API key for YouTube causes crash #121

Nezteb opened this issue Dec 28, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@Nezteb
Copy link

Nezteb commented Dec 28, 2023

Describe the bug
When running the docker run command from the docs to generate a config, this happens:

❯ docker run --rm -it \
    -v .:/app/data \
    ghcr.io/dmunozv04/isponsorblocktv \
    --setup-cli
Could not load config file
Blank config file created
Welcome to the iSponsorBlockTV cli setup wizard
Paired with 0 Device(s). Add more? (y/n) y
Enter pairing code (found in Settings - Link with TV code): REDACTED
Pairing...
Paired device: YouTube on TV
Paired with 1 Device(s). Add more? (y/n) n
API key only needed for the channel whitelist function. Add it? (y/n) y
Get youtube apikey here: https://developers.google.com/youtube/registering_an_application
Enter your API key: REDACTED
Traceback (most recent call last):
  File "/app/main.py", line 3, in <module>
    helpers.app_start()
  File "/app/iSponsorBlockTV/helpers.py", line 128, in app_start
    config_setup.main(config, args.debug)
  File "/app/iSponsorBlockTV/config_setup.py", line 59, in main
    config["apikey"] = apikey
    ~~~~~~^^^^^^^^^^
TypeError: 'Config' object does not support item assignment

Expected behavior

No crash? 😅

iSponsorBlockTV server (please complete the following information):

  • OS: macOS 14.2 (arm64)

Apple TV (please complete the following information):

  • Device: Sony Bravia 4K VH21
  • OS: Android TV OS 10
    • Kernel: 4.19.75 #1 Thu Nov 2 10:49:42 JST 2023
    • Build: BRAVIA_VH21_UC-user 10 QTG3.200305.006.S362 674801 release-keys

Screenshots

My Google Cloud console credentials page:
screenshot_2023-12-28_at_17 30 03@2x

@Nezteb Nezteb added the bug Something isn't working label Dec 28, 2023
@dmunozv04
Copy link
Owner

Hi there, thanks for reporting the issue. I've managed to fix it and will push a new version soon. In the meantime, you could manually edit the config.json file that will be created on the data_dir directory or use the TUI setup wizard, which shouldn't have this problem (replace --setup-cli with --setup on your docker run command

@Nezteb
Copy link
Author

Nezteb commented Dec 28, 2023

Thanks for the quick response David, and for the tip!

In my case I don't actually need the whitelist functionality, so this isn't a blocker for me. I just wanted to report it. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants