[denonmarantz] Improve discovery

[jlaur]

This contains a few discovery improvements:

• Older Denon receivers can now be discovered through UPnP.
• The false assumption about relation between MAC address and brand name (Denon/Marantz) has been replaced by string matching of the am property. This fixes the vendor property being set to Marantz for Denon receivers.
• Label is shortened, i.e. "Denon AVC-X4800H" rather than "Denon AVC-X4800H (Marantz Denon AVC-X4800H)".

The UPnP discovery is now in sync with the suggestion add-on definition (see #16035):

openhab-addons/bundles/org.openhab.binding.denonmarantz/src/main/resources/OH-INF/addon/addon.xml

Lines 12 to 20 in 229c2b7

	<discovery-method>
	<service-type>upnp</service-type>
	<match-properties>
	<match-property>
	<name>manufacturer</name>
	<regex>(?i)DENON</regex>
	</match-property>
	</match-properties>
	</discovery-method>

I have only been able to test this with a very old and very new Denon receiver. If the new assumption about brand string (Denon/Marantz) being part of the model name does not hold for all receivers, the vendor property may not be set.

The mDNS name used for the label should be safe as already described as containing both vendor and model ("Marantz SR5008"):

openhab-addons/bundles/org.openhab.binding.denonmarantz/src/main/java/org/openhab/binding/denonmarantz/internal/discovery/DenonMarantzDiscoveryParticipant.java

Lines 47 to 54 in 229c2b7

	/**
	* Match the serial number, vendor and model of the discovered AVR.
	* Input is like "0006781D58B1@Marantz SR5008._raop._tcp.local."
	* A Denon AVR serial (MAC address) starts with 0005CD
	* A Marantz AVR serial (MAC address) starts with 000678
	*/
	private static final Pattern DENON_MARANTZ_PATTERN = Pattern
	.compile("^((?:0005CD|000678)[A-Z0-9]+)@(.+)\\._raop\\._tcp\\.local\\.$");

Discovery results for my receivers (first is mDNS, second is UPnP):

Resolves #16690

[lsiepel]

LGTM, i'll wait some days for @jwveldhuis to respond.

[jlaur]

I found and fixed some issues when discovering newer receivers through UPnP. Also noticed a problem which I think is within JUPnP and needs further investigation:

Logged:

2024-04-29 22:28:08.464 [DEBUG] [DenonMarantzUpnpDiscoveryParticipant] - Discovered Denon AVC-X4800H, but base URL is missing
2024-04-29 22:28:08.465 [DEBUG] [DenonMarantzUpnpDiscoveryParticipant] - Discovered DENON AVR- 3808
2024-04-29 22:28:08.466 [DEBUG] [DenonMarantzUpnpDiscoveryParticipant] - Discovered Denon AVC-X4800H Aios 6.0V, but base URL is missing

So the AVC-X4800H is not discovered through UPnP even though it seems to me it should be. In UPnP Explorer I see four nodes with everything needed combined, but createResult is only called twice, and in both cases without Base URL and Serial Number.

For this PR it doesn't matter since this receiver is already discovered through mDNS, and the discovery of my old receiver works fine. However, I'm curious why it doesn't fully work as expected and might dig further into this later.