forked from mstinaff/PMS_Updater
-
Notifications
You must be signed in to change notification settings - Fork 5
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
rre #2
Open
luluhoc
wants to merge
42
commits into
luluhoc:master
Choose a base branch
from
mstinaff:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
rre #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
It should be noted that this endpoint does not seem to be returning any data yet: ``` $ curl -i https://plex.tv/api/downloads/5.json HTTP/2 200 date: Thu, 14 Feb 2019 16:56:33 GMT content-type: application/json; charset=utf-8 x-ua-compatible: chrome=1 cache-control: no-cache vary: Origin x-request-id: a62d5e59-c4e9-4398-bedc-42aa28745b7e x-runtime: 0.007216 strict-transport-security: max-age=0 referrer-policy: origin-when-cross-origin x-content-type-options: nosniff x-frame-options: DENY x-xss-protection: 1; mode=block {"computer":{},"nas":{}}% ``` Closes #29
Use new endpoint for Plex v >=1.15
Add dependencies to README.md
Script update for iocage jails
I run this as root on the freenas box itself ssh'd in, i start a screen session and then issue the following command where i have the script saved ./newPMS_Updater.sh -a -r -v -c mycredsfile #set your jail name as seen from iocage list here, and set your zfs storage path for your jail root #where the jail names reside in, there is probably a clever way to pull this automatically from freenas #with sqlite but I haven't investigated that yet. Could also list out the iocage
added libpython symlink, doesn't seem to do anything that I can tell but kept with what original script seems to be doing.
Auto derive the jail path, should only need to set the jail name variable and run from the top. Probably want to change AUTOUPDATE to 1, and VERBOSE to 1. Use with cred files
Merged in jithinvijayan's fix for standard plexpass versions
duplicate file
A prior commit on master removed the PMSPATTERN variable, restored and slightly modified it. Removed dependencies for perl and wget by switching to python and fetch respectively. Wget was using an insecure auth method IMO, switched to fetch and now using the auth token directly from the server preferences to pull the download.
Autoset the $PMSLIVEFOLDER to either $PMSPARENTPATH/plexmediaserver or $PMSPARENTPATH/plexmediaserver-plexpass based on the existence of either folder. Should probably add a check to see if that var is empty afterwards and exit if so.
Moved hardset commandline options to top of file. ToDo: Change plextoken sed scripting to XML parser to extract the token value. Currently remove option won't remove the old tar.bz if its actively installing a new version, may want to change that logic to get it all taken care of in one action. Will remove it if run a second time though.
If you use remove and there is no older file to remove it will print an error, added 2>/dev/null to suppress benign error.
Co-Authored-By: Achilleas Pipinellis <axilleas@users.noreply.github.com>
Adjusted webGet function name to webFetch to reflect using a different backend utility to pull down the data as suggested by @axilleas
Minor readability update around commands
Token auth
Had to work around plex python environment not reading my variables, used ln -s to move python27.zip file to python binary's current directory, seems to work great no issues in testing.
Properly defined the pythonpath variable(Thanks JonnyWong16!) Removed the symlink for the pythonlib as well since the path is properly defined now.
Fixed download by working around environment issues
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.