Opened 12 years ago

Closed 3 years ago

#12089 closed Developer Task (Trac EOL)

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@…> 11 years ago.
UpdateShutdownLock.patch.v1 (4.9 KB ) - added by Bill Meek <keemllib@…> 11 years ago.
Unlock settings on error, thanks to ikke-t on the MAF issues list

Download all attachments as: .zip

Change History (12)

comment:1 by info@…, 11 years ago

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

comment:2 by stuartm, 11 years ago

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 11 years ago by stuartm (previous) (diff)

comment:3 by danrollo@…, 11 years ago

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

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

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.

by Bill Meek <keemllib@…>, 11 years ago

Attachment: UpdateShutdownLock.patch.v0 added

by Bill Meek <keemllib@…>, 11 years ago

Attachment: UpdateShutdownLock.patch.v1 added

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

comment:5 by Stuart Auchterlonie, 10 years ago

Milestone: 0.280.29

comment:6 by Stuart Auchterlonie, 9 years ago

Milestone: 0.2929.0

Milestone renamed

comment:7 by Stuart Auchterlonie, 8 years ago

Milestone: 29.029.1

comment:8 by Stuart Auchterlonie, 8 years ago

Milestone: 29.10.28.2

Moving remaining open tickets to 0.28.2 milestone

comment:9 by Stuart Auchterlonie, 8 years ago

Milestone: 0.28.229.2

Moving remaining open tickets to 29.2 milestone

comment:10 by Stuart Auchterlonie, 3 years ago

Resolution: Trac EOL
Status: newclosed

We have moved all bug tracking to github [1]

If you continue to have this issue, please open a new issue at github, referencing this ticket.

[1] - https://github.com/MythTV/mythtv/issues

Note: See TracTickets for help on using tickets.