Opened 10 years ago

Closed 2 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@…> 9 years ago.
UpdateShutdownLock.patch.v1 (4.9 KB) - added by Bill Meek <keemllib@…> 9 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 Changed 10 years ago by info@…

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

comment:2 Changed 10 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 10 years ago by stuartm (previous) (diff)

comment:3 Changed 9 years ago by danrollo@…

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

comment:4 Changed 9 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 9 years ago by Bill Meek <keemllib@…>

Attachment: UpdateShutdownLock.patch.v0 added

Changed 9 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 8 years ago by Stuart Auchterlonie

Milestone: 0.280.29

comment:6 Changed 8 years ago by Stuart Auchterlonie

Milestone: 0.2929.0

Milestone renamed

comment:7 Changed 6 years ago by Stuart Auchterlonie

Milestone: 29.029.1

comment:8 Changed 6 years ago by Stuart Auchterlonie

Milestone: 29.10.28.2

Moving remaining open tickets to 0.28.2 milestone

comment:9 Changed 6 years ago by Stuart Auchterlonie

Milestone: 0.28.229.2

Moving remaining open tickets to 29.2 milestone

comment:10 Changed 2 years ago by Stuart Auchterlonie

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.