search

MythTV / mythtv

The official MythTV repository
https://www.mythtv.org
GNU General Public License v2.0
710 stars 345 forks source link

Mediaserver running mythtv/32 is not found by Sony Bravia DLNA discovery #683

Closed mholmqvist closed 1 year ago

mholmqvist commented 1 year ago

What steps will reproduce the bug?

Upgrade MythTV/31 to MythTV/32 will break access from Sony Bravia SMART TV

How often does it reproduce? Is there a required condition?

I have tried upgrade multiple times. Samev behavior on two different Sony Bravia that works fine with /31

What is the expected behaviour?

Expected behaviour is that MediaServer is visible also when running version 32.

What do you see instead?

MythTV server /32 is greyed out.

Additional information

I have debugged the DLNA / SSDP handshake and compared MythTV/31 adn MythTV/32 behavior.

I find a typing error in a XML soap namespace declaration. an extra s xmlns:s="htstp://schemas.xmlsoap.org/soap/envelope/"

It is in file mythtv/libs/libmythupnp/httprequest.h And as i can see it as introduced @ commit: https://github.com/MythTV/mythtv/commit/83e9b26f6a06e41425b4a23060d92e4ad4633ba9

Version 31:

HTTP/1.1 200 OK Accept-Ranges: bytes Connection: Keep-Alive Content-Language: en-us Content-Length: 344 Content-Type: text/xml; charset="UTF-8" Date: Mon, 26 Dec 2022 10:36:33 GMT EXT: Keep-Alive: timeout=10 Server: MythTV/31.0+fixes.202203040619.7e4ce1ba98~ubuntu22.04.1 Linux/5.15.0-56-generic UPnP/1.0 transferMode.dlna.org: Interactive

<?xml version="1.0" encoding="utf-8"?>

</u:GetSortCapabilitiesResponse> </s:Body> </s:Envelope>

Version 32:

HTTP/1.1 200 OK Accept-Ranges: bytes Connection: Keep-Alive Content-Language: en-us Content-Length: 345 Content-Type: text/xml; charset="UTF-8" Date: Mon, 26 Dec 2022 10:36:34 GMT EXT: Keep-Alive: timeout=10 Server: MythTV/32.0+fixes.20220325.f69ce764b7-0ubuntu1 Linux/5.15.0-56-generic UPnP/1.0 transferMode.dlna.org: Interactive

<?xml version="1.0" encoding="utf-8"?>

</u:GetSortCapabilitiesResponse> </s:Body> </s:Envelope>

mholmqvist commented 1 year ago

Created Pull request to correct https://github.com/MythTV/mythtv/pull/684

mholmqvist commented 1 year ago

I have verified this issue on the ubuntu jammy packages and it works for me. /Mats

xxx@Mythbuntu2:~# sudo apt policy mythtv-backend mythtv-backend: Installed: 2:32.0+fixes.202212280105.1efbcb1493~ubuntu22.04.1 Candidate: 2:32.0+fixes.202212280105.1efbcb1493~ubuntu22.04.1 Version table: *** 2:32.0+fixes.202212280105.1efbcb1493~ubuntu22.04.1 500 500 https://ppa.launchpadcontent.net/mythbuntu/32/ubuntu jammy/main amd64 Packages 100 /var/lib/dpkg/status 2:32.0+fixes.20220325.f69ce764b7-0ubuntu1 500 500 http://se.archive.ubuntu.com/ubuntu jammy/multiverse amd64 Packages 2:31.0+fixes.202203040619.7e4ce1ba98~ubuntu22.04.1 500 500 https://ppa.launchpadcontent.net/mythbuntu/31/ubuntu jammy/main amd64 Packages