dresden-elektronik / deconz-rest-plugin

deCONZ REST-API plugin to control ZigBee devices
BSD 3-Clause "New" or "Revised" License
1.9k stars 502 forks source link

Manufacturername matching is still case sensitive, still requiring individual spelling in DDF #7965

Open SwoopX opened 2 weeks ago

SwoopX commented 2 weeks ago

Does the issue really belong here?

Is there already an existing issue for this?

Describe the bug

With https://github.com/dresden-elektronik/deconz-rest-plugin/pull/5958/commits/f9bd82f68bf6ac4f3c09726dbcafc4f8242b4033, DDF manufacturername matching becamse case insensitive to ease up the situation with differenct spellings of manufacturernames (see also https://github.com/dresden-elektronik/deconz-rest-plugin/pull/5903#issuecomment-1096797995). However, due to some internal deconz changes, his functionality got lost again. The anticipated fix (https://github.com/dresden-elektronik/deconz-rest-plugin/pull/7864) does apparently not restore an insensitive check.

Similar behavior discussed in #7949.

Steps to reproduce the behavior

  1. Take a device you know the manufacturername of (exact spelling), which is NOT paired to deconz. Must have a valid DDF available
  2. Modify the DDFs manufacturername and change capitalization of an arbitrary letter. Ensure the DDF does not contain a fitting manufacturername, spelling-wise.
  3. Try to pair the device and watch it fail

Expected behavior

Manufacturername should be matched case-insensitive.

Screenshots

grafik

Environment

deCONZ Logs

13:08:16:891 DEV no DDF for 0xCCCCCCFFFE393896, modelId: eTRV0100

Additional context

No response