-
Notifications
You must be signed in to change notification settings - Fork 31
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
JideTech ONVIF Camera - #24
Comments
Hi, specification. message.data may be array I fix it make from simpleItem array to object like {$.Name, $.Value} |
@ksupipr I looked at your branch and didn't see any changes or pull requests. Am I missing something? Thanks |
Holidays ) |
@jdesai61 https://github.com/ksupipr/onvif2mqtt/tree/bugfix/issue24 I think what value send to the event, for compatibility with old projects. |
Resolved with #34 . Reopen if not, thanks! |
I was playing around with my JideTech cameras and notices that the "simpleItem" they send in motion event is an array (see debug below)
Is this allowed in ONVIF standard? If so, it will be nice to have a fix. For now, I hacked SubscriberGroup.js line from
to
And it worked for my camera - obviously this breaks other cameras that don't send an array. Perhaps a check on testing for array could be added as a fallback? Thanks
The text was updated successfully, but these errors were encountered: