UPNP don't notify embedded child devices by default

[andrewfg]

See #4712

Signed-off-by: Andrew Fiddian-Green software@whitebear.ch

[andrewfg]

Ping @lolodomo ..

Note: I wrote the code, but I do not have any time for testing, so can you please have a look at it yourself.

[andrewfg]

@lo92fr if your addon discovery service participant wants to be notified about embedded child devices then it will have to override the notifyChildDevices() method.

[mherwege]

See my question: #4712 (comment)

[andrewfg]

@mherwege either we have to modify..

• some bindings to prevent them seeing unwanted children (e.g. Sonos) (opt out model), or
• other bindings to explicitly allow them to see wanted children (opt in model)

On balance I think the opt in model is safer.

[mherwege]

@mherwege either we have to modify..

• some bindings to prevent them seeing unwanted children (e.g. Sonos) (opt out model), or
• other bindings to explicitly allow them to see wanted children (opt in model)

On balance I think the opt in model is safer.

For upnpcontrol, which is a generic binding, you would want to see the embedded devices. That would mean the Sonos embedded media server and renderer would be discovered. I believe that is what @lolodomo wanted to prevent. It is easy enough to not have them discovered by the Sonos binding (opt-in or opt-out doesn’t matter). But you can’t avoid discovering it with the upnpcontrol binding.

[andrewfg]

@mherwege I think you misunderstand this. The OH Core service will ALWAYS discover both root and embedded devices. The bindings respective discovery participants will by default not receive the NOTIFICATIONS for embedded devices. And it you want your addon's disco participant to receive those notifications then you need to make a one liner override of its notifyChildDevices() method.

[mherwege]

@mherwege I think you misunderstand this. The OH Core service will ALWAYS discover both root and embedded devices. The bindings respective discovery participants will by default not receive the NOTIFICATIONS for embedded devices. And it you want your addon's disco participant to receive those notifications then you need to make a one liner override of its notifyChildDevices() method.

I think I understand this well enough. Again: you would want to override this for the upnpcontrol binding because that binding would want to see the embedded devices for many devices (e.g. Denon has their media player as an embedded device). Therefore it will discover the embedded Sonos media server and media renderer. You cannot avoid that for upnpcontrol. That’s why I asked @lolodomo what binding discovered these embedded devices. Was it the sonos binding or the upnpcontrol binding?

[lolodomo]

That’s why I asked @lolodomo what binding discovered these embedded devices. Was it the sonos binding or the upnpcontrol binding?

The Sonos binding.
The upnpcontrol binding is not installed.