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

Fix device appearing offline #24

Merged
merged 1 commit into from
Jun 16, 2018
Merged

Conversation

behinddesign
Copy link
Contributor

@behinddesign behinddesign commented Jun 15, 2018

These changes seem to keep the device marked as online once it's added and in most cases allows for the device to be automatically found.

It does seem however that Plex does act strangely sometimes, but once a device has been added, it appears to remain online.

It is recommended to delete the existing device so the new UUID is marked with this device, along with the SSDP packets being marked with the correct UUID fixes most problems. Other adjustments have been inspired by https://github.com/tombowditch/telly , such as the name of the device and baseurl in the device.xml.

Restarting Plex once a device has been added seems to be required

ControlURL and EventSubURL adjustments have an unknown result, it appears plex never calls them anyway. I have left them in there but they could probably be reverted. It however didn't make sense to revert them as the endpoint didn't match the URL

… HDHomerun. Add ENV to set device uuid

Update device.js

Try with dashes

Update router.js

Seeing if adding .xml works

Update ssdp.js

Try adding uuid to the server

Add UID handling, BaseURL and rename the device so it's prefixed with HDHomerun. Add ENV to set device uuid
@behinddesign behinddesign changed the title Add UID handling, BaseURL and rename the device so it's prefixed with… Fix device appearing offline Jun 15, 2018
@behinddesign behinddesign mentioned this pull request Jun 15, 2018
@jfarseneau jfarseneau merged commit 167b043 into jfarseneau:master Jun 16, 2018
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