search

RandomNinjaAtk / docker-sonarr-extended

Sonarr (develop) with bash scripts to automate and extend functionality.
GNU General Public License v3.0
66 stars 15 forks source link

[Question] Plex Notify Script Update? #12

Closed justinglock40 closed 1 year ago

justinglock40 commented 1 year ago

Afternoon. I just wanted to inquire whether you'd be updating your Plex Notify script to use the API scan instead of the CLI Scan since Plex says that will be deprecated in a future undisclosed version?

Thanks for what we have available now! Works great!!

RandomNinjaAtk commented 1 year ago

This is the first I am hearing this, do you have any links to information on it?

If I can make the change, I will...

RandomNinjaAtk commented 1 year ago

After some quick research, I'm not really understanding what your getting at because this script already uses the API... Based on the information I can find...

Script uses this information to update and send commands: https://support.plex.tv/articles/201638786-plex-media-server-url-commands/

So if your not talking about that, you'll need to provide additional information and documentation...

Edit: Also, where did you find an announcement that this is being deprecated?

justinglock40 commented 1 year ago

Alright so I figured out what was happening. You're 100% correct. Your script is already using the API operations.

I had the older Plex Autoscan set to run a cron job once a day to catch things that were missed during scans requests.

So that uses the Plex CLI Scanner that triggers a warning in the log that says "--scan is deprecated and will be removed in future versions of Plex Media Server."

So because I forgot about that cron job that I was attributing that warning message to your script which was incorrect on my part.

So please close and disregard this. Thank you!

justinglock40 commented 1 year ago

But this is the message you get from the command line scanner.

image

RandomNinjaAtk commented 1 year ago

Have you actually reviewed the script code?

That's not what this script uses, see here: https://github.com/RandomNinjaAtk/docker-sonarr-extended/blob/d91667ac52de01adb242133853472712197179aa/root/scripts/PlexNotify.bash#L91

So I don't think that is relevant.... And you haven't provided documentation for the alternative method....

justinglock40 commented 1 year ago

Yea we're on the same page. Your script was not causing this message.

It was an older script from someone else that I had running as a cron job that was triggering the message.

So, yes, this is irrelevant.

RandomNinjaAtk commented 1 year ago

@justinglock40 Sorry for confusion with my last message, I only saw your last comment, not the one before it where you'd already figured it out and said the same thing...