fix: the responses to several requests returns erroneous values

[FelixPetriconi]

The comparison of the several responses with DLNA complaint servers has shown differences and were fixed accordingly

Please bare with me but I don't know much about C# or the used frameworks. Please advise how to improve the code if necessary.

I just can tell that now my Panasonic Blu-Ray player is able to connect to the Jellyfin instance.
The display of photos, streaming of mp3 and flac, and some mpg works fine.
I intent to add a working profile for that player as soon as I have found out how to get .mkv container playing.

[DSLinuxUser]

Tried today the updates and now the Panasonic DMR-BCT750/755 is able to connect to JellyFin and plays videos back.

[FelixPetriconi]

Which profile did you use? What kind of videos container did you play back?

[crobibero]

I don't see how this would work as the response isn't being set to anything.

[FelixPetriconi]

I cannot give you an answer to this, because I don't much about the used framework. I only can say that a response that is set to any mime type is a) not been accepted by the Panasonic player and b) it has now with this changes same result as with e.g. Emby 4.8.10 and Twonky Server 8.5.2.
I checked it on protocol level with Wireshark.

[DSLinuxUser]

Which profile did you use? What kind of videos container did you play back?

What kind of profile do you mean? Device profile?
If device profile, Jellyfin tells me in the log, that the default profile is selected.

Test was done MP4 with H264/AAC.

I don't see how this would work as the response isn't being set to anything.

Well, before this patch, the Panasonic would just throw the error "Unable to connect to server".
With this patch, it just works.

[FelixPetriconi]

The same for me now.

[DSLinuxUser]

Applied the changes, recompiled and replaced the DLLs and the Panasonic can still access JellyFin.

[disgustipated]

It looks like this is taking the response and adding/modifying some headers to the packet. Is that correct? What is the SID? Theres a System import that I'm not sure is needed. I'll test this out this week to see if I can get other devices to work as expected too.

[FelixPetriconi]

I don't know much about Jellyfin or the DLNA plugin but I assume it is a session ID that is generated by the DLNA plugin and that helps to associate future request of that DLNA client to a certain session context.

[zznty]

Thank you, building from this pr has fixed jellyfin discovery in both vlc and good old lg smart tv.

[FelixPetriconi]

Are there problems with this PR? If so, please let me know.
Keeping PRs that long open cause automatically problems. I just solved the new conflicts.

[DSLinuxUser]

Are there problems with this PR? If so, please let me know. Keeping PRs that long open cause automatically problems. I just solved the new conflicts.

For me these changes fixed the issue and I approved the PR.
At least I think I did:

[Shadowghost]

I wonder if these modifications should not be moved to the handler itself instead of the controller

[ypopovych]

Hello.

Will this PR be merged?

[crobibero]

Releasing for 10.10 isn't an easy task when 10.11 is in RC phase. As such I've created a PR with these commits along with some cleanup to #166 and merged into unstable.