Opened 5 years ago

Last modified 11 months ago

#12089 new Developer Task

Streaming from UPNP doesn't block shutdown

Reported by: stuartm Owned by: stuartm
Priority: minor Milestone: 29.2
Component: MythTV - UPnP Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Streaming via upnp (or any other backend service) should prevent backend shutdown.

Attachments (2)

UpdateShutdownLock.patch.v0 (4.8 KB) - added by Bill Meek <keemllib@…> 4 years ago.
UpdateShutdownLock.patch.v1 (4.9 KB) - added by Bill Meek <keemllib@…> 4 years ago.
Unlock settings on error, thanks to ikke-t on the MAF issues list

Download all attachments as: .zip

Change History (11)

comment:1 Changed 5 years ago by info@…

Using AirPlay? should prevent shutdown as well. "Please don't stop the music" :)

comment:2 Changed 4 years ago by stuartm

One of the issues with addressing this for UPnP is that clients can and do buffer much of the file they are streaming, especially with music this can be the whole file. This may mean the backend has no way of knowing that the client is still in playback and that it needs to stay awake.

Secondly, we should include use of the other http interfaces - WebFrontend? and services API as activities that block shutdown or which reset the back idle timer. However how do we distinguish between user driven activity and automatic polling by a client application.

I don't have answers to these questions yet.

Last edited 4 years ago by stuartm (previous) (diff)

comment:3 Changed 4 years ago by danrollo@…

Too enable workarounds, could a "preventShutdown" method be added to the service API?

comment:4 Changed 4 years ago by Bill Meek <keemllib@…>

Patch attached. Tested on v0.28-pre-2422-g8e6678a-dirty.

Proposed solution: Myth/UpdateShutdownLock??Increment=<x>. Where x will typically be 1 or -1. The return will be an int with -1 for all failures, otherwise, the value of the setting MythShutdownLock?. POST required since data is changed. Failures will appear in the BE log.

Changed 4 years ago by Bill Meek <keemllib@…>

Attachment: UpdateShutdownLock.patch.v0 added

Changed 4 years ago by Bill Meek <keemllib@…>

Attachment: UpdateShutdownLock.patch.v1 added

Unlock settings on error, thanks to ikke-t on the MAF issues list

comment:5 Changed 3 years ago by Stuart Auchterlonie

Milestone: 0.280.29

comment:6 Changed 3 years ago by Stuart Auchterlonie

Milestone: 0.2929.0

Milestone renamed

comment:7 Changed 13 months ago by Stuart Auchterlonie

Milestone: 29.029.1

comment:8 Changed 11 months ago by Stuart Auchterlonie

Milestone: 29.10.28.2

Moving remaining open tickets to 0.28.2 milestone

comment:9 Changed 11 months ago by Stuart Auchterlonie

Milestone: 0.28.229.2

Moving remaining open tickets to 29.2 milestone

Note: See TracTickets for help on using tickets.