homebridge-plugins / homebridge-august

The Homebridge August plugin allows you to access your August/Yale Locks from HomeKit with Homebridge.
ISC License
36 stars 9 forks source link

Bug: Log reports error reading properties, Home.app then reports lock "was jammed" #16

Closed hrhnick closed 2 years ago

hrhnick commented 2 years ago

Describe The Bug

Home.aLog reports error reading properties, Home.app then sends notification lock "was jammed"pp reports lock "was jammed" despite lock not being jammed. August app and other plugins do not report jam.

To Reproduce

Error is random with no seeming correlation to time since lock or unlock.

Expected behavior

No error, no Home.app notification.

Relevant log output

[10/8/2022, 6:26:09 PM] [August] TypeError: Cannot read properties of undefined (reading 'retryCount')
[10/8/2022, 6:26:09 PM] [August] Lock: Front Door failed lockStatus (refreshStatus), Error Message: "Cannot read properties of undefined (reading 'retryCount')"

Config for homebridge-august

"name": "August",
            "credentials": {
                "augustId": "xxxxx@xxxxxx.com",
                "password": "xxxxxx",
                "validateCode": "xxxxx",
                "notice": "Keep your tokens a secret!"
            },
            "options": {
                "logging": "standard"
            },
            "platform": "August"

Screenshots

IMG_B63930AD8B1F-1

Device & Model

AugusAugust WiFi Smart Lock

Node.js Version

16.17.1

NPM Version

8.5.0

Homebridge Version

oznu/homebridge:latest

Homebridge August Plugin Version

4.50.1

Homebridge Config UI X Plugin Version

1.0.4

Operating System

Docker

donavanbecker commented 2 years ago

@hrhnick, currently if I get a retry count I report jammed. I think I will remove that code.

3ne commented 2 years ago

Seeing the same error.

hrhnick commented 2 years ago

@donavanbecker could this somehow trigger the device to try unlock and relock? (To clear the jam?)

I can’t be sure but I am pretty certain I was seeing some odd unlocks as well but couldn’t find it in the logs.

donavanbecker commented 2 years ago

@hrhnick @3ne, lets try v1.0.5

3ne commented 2 years ago

@hrhnick @3ne, lets try v1.0.5

Updating.

mojodean commented 2 years ago

From my experience, the update has cleaned up the jam alerts, and I'm getting normal lock/unlock alerts. I was having the same issues OP was.

donavanbecker commented 2 years ago

JAMMED should only happen if the API sends a false for both Locked and Unlocked.

3ne commented 2 years ago

@donavanbecker all looks good ever here as well.

donavanbecker commented 2 years ago

Good to hear!