[denonmarantz] Wrong brand is set in vendor property

[jlaur]

I just purchased a Denon AVC-X4800H to replace my beloved old Denon AVR-3808, and I noticed the wrong brand being set in the vendor property:

Serial number is the same as the MAC address, which starts with 00:06:78.

My old 3808 recievers's MAC address starts with 00:05:CD.

Expected Behavior

If vendor can be determined reliably, it should be set. Otherwise the property should not be set.

Current Behavior

The vendor is set based on false assumptions about MAC addresses:

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

Lines 47 to 64 in 321fde5

	/**
	* 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\\.$");
	/**
	* Denon AVRs have a MAC address / serial number starting with 0005CD
	*/
	private static final String DENON_MAC_PREFIX = "0005CD";
	/**
	* Marantz AVRs have a MAC address / serial number starting with 000678
	*/
	private static final String MARANTZ_MAC_PREFIX = "000678";

and set here:

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

Lines 87 to 95 in 321fde5

	* The Vendor is not available from the mDNS result.
	* We assign the Vendor based on our assumptions of the MAC address prefix.
	*/
	String vendor = "";
	if (serial.startsWith(MARANTZ_MAC_PREFIX)) {
	vendor = "Marantz";
	} else if (serial.startsWith(DENON_MAC_PREFIX)) {
	vendor = "Denon";
	}

The assumption might have been correct in the past, but it seems that Sound United is now sharing more technology between the two brands, so it can no longer be assumed.

Possible Solution

Determining vendor should not be based on MAC address. It seems there is enough mDNS information to determine it:

I need to get my old 3808 online, and will check the mDNS result for this one as well.

Your Environment

• Version used: 4.2 snapshot

[jlaur]

Additionally, the label is created like this: "Denon AVC-X4800H (Marantz Denon AVC-X4800H)"

[jlaur]

It seems old my Denon AVR-3808 can not be found through mDNS at all, only UPnP (screenshot from Device Spy):

As text:

• Base URL http://192.168.0.44:8080/
• Device icon Present, 48x48
• Device URN urn:schemas-upnp-org:device:MediaRenderer:1
• Embedded devices 0
• Expiration timeout 1800
• Friendly name Network Audio
• Has presentation True
• Interface to host 192.168.0.3
• Manufacturer DENON
• Manufacturer URL http://www.denon.com
• Model description AV SURROUND RECEIVER
• Model name AVR-3808
• Model number 3808
• Model URL http://www.denon.com/
• Presentation URL http://192.168.0.44:8080/index.html
• Remote endpoint 192.168.0.44:8080
• Serial number 0005CDxxx
• Services 4
• Standard type MediaRenderer
• Version 1.0

[jlaur]

Related: #16035