Skip to content

ARLChecker Initial Release #187

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

Merged
merged 18 commits into from
Feb 10, 2024
Merged

Conversation

hockeygoalie35
Copy link
Contributor

Initial release to ARLChecker, addresses #133 .

ARLChecker.py:
Python script checks the validity of ARL token set in extended.conf. If the token comes back invalid, a (user-created) Telegram bot alerts them, and prompts for a new one. The bot functionality is disabled by default. A wiki entry will be added on how to create an Telegram bot. The user can scan the logs or run the checker manually to understand the current status (wiki entry to be made). As a fallback, a text file is updated with the status of the token, so the user can go in check if the suspect something's wrong. I would like to at some point leverage the Lidarr API to notify the user if that's possible.

ARLChecker (bash)
Bash script that runs ARLChecker.py with -c flag, then sleeps for interval set in extended.conf (default 24h)

image
Other changes to support this feature:

  • Create new directory 'python' in custom-services.d to prevent py script from auto-running (incorrectly).
  • Added relevant parameters in extended.conf
  • Fix a couple typos in extended.conf
  • Add to Read.me ARLChecker as feature.
  • Add colorama and python-telegram-bot to pip install in setup.bash
  • Add auto-download of ARLChecker Bash and .py to setup.bash

Future improvements:

  • Better extended.conf parsing
  • Better notification without relying on logs or Telegram bot if user chooses not enable it.
  • General cleanup. I don't like the way the original Deezer login class works.
  • Wiki entries for this functionality and Telegram bot creation.

Feedback and suggestions welcome.

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

Successfully merging this pull request may close these issues.

2 participants