Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Device not working anymore because of wrong endpoint #4131

Closed
TilmanK opened this issue Aug 19, 2020 · 2 comments
Closed

Device not working anymore because of wrong endpoint #4131

TilmanK opened this issue Aug 19, 2020 · 2 comments

Comments

@TilmanK
Copy link
Contributor

TilmanK commented Aug 19, 2020

I upgraded from 1.14.1 to 1.14.3. This broke my living room lamp. I switched between both git tags and noticed that this is reproducible. The reason can be seen when sniffing traffic.

When sending a {state: "on"} to the device, 1.14.1 sends it to the correct endpoint:

ZigBee Application Support Layer Data, Dst Endpt: 2, Src Endpt: 1
Frame Control Field: Data (0x00)
Destination Endpoint: 2
Cluster: On/Off (0x0006)
Profile: Home Automation (0x0104)
Source Endpoint: 1
Counter: 213

Doing the same with 1.14.3 uses 1 as destination endpoint which is wrong:

ZigBee Application Support Layer Data, Dst Endpt: 1, Src Endpt: 1
Frame Control Field: Data (0x00)
Destination Endpoint: 1
Cluster: On/Off (0x0006)
Profile: Home Automation (0x0104)
Source Endpoint: 1
Counter: 25

The root cause lies in this bug: #4000
With this change #4000

The device is not the same. You just captured my device and changed the identifier by removing the space. :)

@TilmanK
Copy link
Contributor Author

TilmanK commented Aug 19, 2020

I'll submit a PR for this...

Koenkk added a commit to Koenkk/zigbee-herdsman-converters that referenced this issue Aug 19, 2020
….57 (#1487)

* Reintroduce Paul Neuhaus 100.110.39 to work side-by-side with 100.110.57 #Koenkk/zigbee2mqtt#4131

* Fixed lint

* Update devices.js

Co-authored-by: Koen Kanters <koenkanters94@gmail.com>
@Koenkk
Copy link
Owner

Koenkk commented Aug 19, 2020

Merged, sorry for breaking! Assuming this can be closed now.

@Koenkk Koenkk closed this as completed Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants