samemory / homebridge-eufy-security

Work in progress
Apache License 2.0
97 stars 14 forks source link

[enhancement]: include homekit secure video #337

Closed adekka closed 2 years ago

adekka commented 2 years ago

Describe the solution you'd like

In homebridge 1.4.0 HSV is included. The request is to include HSV in this plugin.

github-actions[bot] commented 2 years ago

Did you check this Road Map pages ?

lenoxys commented 2 years ago

hey, thanks ! Yes thinking of that too. I'm waiting 1.4.0 going GA before going deeper on that point.

fullphat commented 2 years ago

Just upgraded to 1.4 and have started using camera.ui for one of my devices. Works brilliantly, so a massive +1 from me too on incorporating HSV into this plugin. šŸ‘šŸ¼

technikosoldat commented 2 years ago
schliemann commented 2 years ago

@fullphat

I don't think we can take credit. We haven't changed our codebase - assuming that is necessary? How does it work and are something missing?

adekka commented 2 years ago

homebridge-camera-UI has included HSV in the 5.0 version. Perhaps this is a good start?

steverae68 commented 2 years ago

Just upgraded to 1.4 and have started using camera.ui for one of my devices. Works brilliantly, so a massive +1 from me too on incorporating HSV into this plugin. šŸ‘šŸ¼

I have also switched from ffmpeg to camera.ui for the HKSV. Works a treat, although I did find I had to disable the video analysis as my pi was melting lol

Hope to get HKSV soon so I can get rid of the native eufy app - one less thing to worry about

adekka commented 2 years ago

I'm not the programmer but a good researcher ;) It is also implemented in this code

This is mentioned in how it works: I won't get into the spec, you can read that. Basically HAP-NodeJS was modified to request the recording options available for a device and create the appropriate RecordingManagement that communicates with HomeKit. When motion is triggered, HomeKit dials in to open a data connection with HAP-NodeJS. HAP-NodeJS will invoke a callback for the plugin to generate mp4 fragments. Creating those fragments properly is up to the plugin. HAP-NodeJS will then properly chunk and stream the fragments to the Home hub.

adekka commented 2 years ago

and more info on the implementation of HSV with unofficial support specification

steverae68 commented 2 years ago

Oh I'm so looking forward to this coming for my doorbell... this is the last piece of the puzzle then everything I have is connected through HKSV.

Still don't understand why eufy can enable their 2C cams for HKSV but not their doorbell

fullphat commented 2 years ago

Still don't understand why eufy can enable their 2C cams for HKSV but not their doorbell

It's the same with the original floodlight camera. That doesn't support RTSP either. I wonder if someone else actually makes some of these products and they don't have the onboard chipset or memory to support these features. Very frustrating.

steverae68 commented 2 years ago

Yeah, very frustrating indeed. If eufy had enabled HK support for their doorbell, they could have had a huge market advantage, leading to lots more salesā€¦..

Anyway, good that homebridge is thereā€¦ Letā€™s hope the guys behind this awesome plugin can do what eufy themselves seem to not want to do.

As always, happy to test any betaā€¦ currently running latest beta with local storage - works great, but testing cameraUI with HKSV is soooooo much betterā€¦.

Honestly Iā€™m super impressed

Sent from my iPhone

On 26 Jan 2022, at 18:31, fullphat @.***> wrote:

ļ»æ Still don't understand why eufy can enable their 2C cams for HKSV but not their doorbell

It's the same with the original floodlight camera. That doesn't support RTSP either. I wonder if someone else actually makes some of these products and they don't have the onboard chipset or memory to support these features. Very frustrating.

ā€” Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

schliemann commented 2 years ago

This is indeed super exiting. This would however require quite some work. Also there are a number of things to consider. If we start a livestream on a motion trigger and send a video to apple - then the video will at minimum be delayed by the amount of time it takes to start a livestream.

An alternative would be to simple send videoclips from the eufy cloud to Apple when they have been captured.

Don't really know what would be ideal solution and I don't have the time to look in to this currently. Right now my focus is on making the snapshots work with local livestream instead of the last event from the eufy cloud.

paqpaqpaq commented 2 years ago

@schliemann to my understanding these are separate actions Streaming would still go through eufy, but analyzing and recording would be handled on the background by homekit. At least that is what I understand using a ring doorbell in scrypted. There is no 'extra' time needed to start the stream, just the notification seems(!) to be a bit lagged.

schliemann commented 2 years ago

Well. I'm only taking about devices that eufy don't offer homekit integration for. Those will have the issues with hksv which I mentioned.

steverae68 commented 2 years ago

Yeah, from my testing using cameraUI running HKSV against my hik vision cctv system using only rtsp - itā€™s insanely fast, much much faster and responsive than normal

Sent from my iPhone

On 27 Jan 2022, at 13:57, paqpaqpaq @.***> wrote:

ļ»æ @schliemann to my understanding these are separate actions Streaming would still go through eufy, but analyzing and recording would be handled on the background by homekit. At least that is what I understand using a ring doorbell in scrypted. There is no 'extra' time needed to start the stream, just the notification seems(!) to be a bit lagged.

ā€” Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

paqpaqpaq commented 2 years ago

Well. I'm only taking about devices that eufy don't offer homekit integration for. Those will have the issues with hksv which I mentioned.

Ring hasn't got any homekit integration. There isn't really any additional delay. It would be awesome if the eufy plugin could get somehow get worked into camera.ui or scrypted, or (the other way around) have the HB plugin working with HKSV.

steverae68 commented 2 years ago

I believe that the homebridge-ffmpeg-camera is being updated to support HKSV too, be interested to se how this works with HKSV integrated, this is what Iā€™ve used for so many years.

CameraUI has a lot of additional unnecessary overheads tbf, but still redicukously quick.

Tbf my hikvision cams through cameraUI respond quicker than my eufy 2c cam that is native with HKSV.

Hereā€™s hoping for an update soon.

Iā€™ve got no issues with the latest beta and local storage but always nice to have everything controlled via HomeKit

Sent from my iPhone

On 27 Jan 2022, at 19:29, paqpaqpaq @.***> wrote:

ļ»æ Well. I'm only taking about devices that eufy don't offer homekit integration for. Those will have the issues with hksv which I mentioned.

Ring hasn't got any homekit integration. There isn't really any additional delay. It would be awesome if the eufy plugin could get somehow get worked into camera.ui or scrypted, or (the other way around) have the HB plugin working with HKSV.

ā€” Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

Dravegon commented 2 years ago

Iā€™m looking forward to have HKSV on my Battery 2K Doorbell too, iā€™ve done some research and understand that for the HKSV to work the Doorbell needs rtsp to work as HKSV with camera.ui Now i found this: https://www.reddit.com/r/EufyCam/comments/nal27f/enabling_rtsp_on_battery_doorbell/

It says it can enable rtsp on the Battery 2K Doorbell by modifying the Eufy android app so the rtsp option can be enabled in settings

Can someone confirm this would enable rtsp so it can be used for HKSV?

steverae68 commented 2 years ago

From my own research, together with the lack of other information regarding rtsp streams from the eufy doorbel cam, I don't believe This is something that is viable. The post mentioned from Reddit is from along time ago.

My interpretation of this is like sticking a 'turbo' badge on the back of you car and expecting it to have a working turbo. Even though the option is exposed in the software, does not mean the feature will work.... it no expert in reverse engineering but as with why eufy have not enabled HKSV natively on the doorbell, leads me to believe the chipset in the physical hardware cannot support this. So any kind of HKSV would be a hack via this plugin. Which is what I'm waiting for.

Just my 2 pennies worth and opinion, as always, here's hoping we can get HKSV soon

fullphat commented 2 years ago

I'm not sure RTSP is a requirement for HKSV; it's more about the accessory supporting additional private HomeKit characteristics. I think, so long as you can get a RTMP stream from ffmpeg, HKSV should be able to pick it up. My floodlight cam doesn't support RTSP, but using this plugin, the P2P stream can be pulled into ffmpeg and sent out as RTMP, so I'm still optimistic this plugin could support HKSV for all camera types.

AxelDreemurr commented 2 years ago

+1! Having HKSV would be the icing on top!

lenoxys commented 2 years ago

Iā€™m looking forward to have HKSV on my Battery 2K Doorbell too, iā€™ve done some research and understand that for the HKSV to work the Doorbell needs rtsp to work as HKSV with camera.ui

Now i found this:

https://www.reddit.com/r/EufyCam/comments/nal27f/enabling_rtsp_on_battery_doorbell/

It says it can enable rtsp on the Battery 2K Doorbell by modifying the Eufy android app so the rtsp option can be enabled in settings

Can someone confirm this would enable rtsp so it can be used for HKSV?

For me just because it's battery device we shouldn't use HKSV on them this will kill their battery. Why not on wired device.

Dravegon commented 2 years ago

Iā€™m looking forward to have HKSV on my Battery 2K Doorbell too, iā€™ve done some research and understand that for the HKSV to work the Doorbell needs rtsp to work as HKSV with camera.ui Now i found this: https://www.reddit.com/r/EufyCam/comments/nal27f/enabling_rtsp_on_battery_doorbell/ It says it can enable rtsp on the Battery 2K Doorbell by modifying the Eufy android app so the rtsp option can be enabled in settings Can someone confirm this would enable rtsp so it can be used for HKSV?

For me just because it's battery device we shouldn't use HKSV on them this will kill their battery. Why not on wired device.

Well, just hookup a (good enough) transformer and it will operate same as a wired doorbell with battery backup for when your transformer would become lazy or hot lol HKSV would introduce another ā€œextraā€ security layer on whoā€™s in front of your house with announcing names by Siri through face recognition(HKSV needed) This would come in handy when either: busy, driving a car(CarPlay) , or just unable to check your phone or watch (charging) So whether itā€™s a known or unknown(burglar) person you then always know who or whatā€™s in front of your door( and yes i donā€™t live in some fancy utopia lol and no, a expensive 24/7 security camera system would be overkill)

Bonus tip: for anyone who thinks you need a HomePod, Apple TV, or iPad for Siri announcements through HKSV, you donā€™t. Just have yourself a Jailbroken iPhone with Homekit Remote Hub Enabler tweak from Limneos on Bigboss repository(free/works on all iOS versions) you are welcome ;-)

steverae68 commented 2 years ago

Further to this, I have a eufy 2c cam which is also battery but with native HKSV support so I canā€™t see ā€˜batteryā€™ being an issue, more than likely a chipset issue on the actual deviceā€¦..

The chipset in the doorbell being different to that in the 2c cams would pose a major hurdle

Still looking forward to this (if possible)

Sent from my iPhone

On 1 Feb 2022, at 10:45, Dravegon @.***> wrote:

ļ»æ Iā€™m looking forward to have HKSV on my Battery 2K Doorbell too, iā€™ve done some research and understand that for the HKSV to work the Doorbell needs rtsp to work as HKSV with camera.ui Now i found this: https://www.reddit.com/r/EufyCam/comments/nal27f/enabling_rtsp_on_battery_doorbell/ It says it can enable rtsp on the Battery 2K Doorbell by modifying the Eufy android app so the rtsp option can be enabled in settings Can someone confirm this would enable rtsp so it can be used for HKSV?

For me just because it's battery device we shouldn't use HKSV on them this will kill their battery. Why not on wired device.

Well, just hookup a (good enough) transformer and it will operate same as a wired doorbell with battery backup for when your transformer would become lazy or hot lol HKSV would introduce another ā€œextraā€ security layer on whoā€™s in front of your house with announcing names by Siri through face recognition(HKSV needed) This would come in handy when either: busy, driving a car(CarPlay) , or just unable to check your phone or watch (charging) So whether itā€™s a known or unknown(burglar) person you then always know who or whatā€™s in front of your door( and yes i donā€™t live in some fancy utopia lol and no, a expensive 24/7 security camera system would be overkill)

Bonus tip: for anyone who thinks you need a HomePod, Apple TV, or iPad for Siri announcements through HKSV, you donā€™t. Just have yourself a Jailbroken iPhone with Homekit Remote Hub Enabler tweak from Limneos on Bigboss repository(free/works on all iOS versions) you are welcome ;-)

ā€” Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

lenoxys commented 2 years ago

when it will be implemented we will provide the choice to enable or not per camera.

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

Dravegon commented 2 years ago

Any news on HKSV implementation?

steverae68 commented 2 years ago

To be fair. HKSV requires a rtsp stream to work. This is not available on the eufy doorbell, I don't believe. Only way this could be done is to re-encode the stream from the camera or have the plugin generate a rtsp stream on the fly to push to HKSV on the fly.

Hmmm

karesake commented 2 years ago

Yes, and Eufy's RTSP stream is unfortunately not good anyway. It is only designed to record events to a NAS and is not intended for permanent live stream. Converting the whole thing now through a plugin would significantly affect performance.

Furthermore, the stream would have to remain active 24/7, which would also have a negative impact on retrieval from other devices. Users of Raspberrys and Docker would be excluded, so you need some power for it. A Mac is then required. And then not a 10-year-old box.

Dakitoo commented 2 years ago

Hallo, ich hoffe das jemand auch deutsch lesen kann.

Ich habe eine Frage. Ich nutze die aktuelle Beta mit meiner Video-TĆ¼rklingel T8210. Ich bekomme es auch im HomeKit angezeigt. Es dauert aber dann beim ƶffnen meist lƤnger. Geht es, dass sich das Video automatisch ƶffnen lƤsst wenn die Klingel betƤtigt wurde? In der Home App kann ich keine Camera auswƤhlen, die geƶffnet werden soll wenn die Klingel gedrĆ¼ckt wurde. Hat das jemand umgesetzt?

Dravegon commented 2 years ago

Rule #1 write English so everybody who reads this can understand(Google Translate before posting) The reason why it usually takes longer to load is because of using Wifi on the Homebase(if not your cables/router is poorly configured) use an ethernet cable connected to your router instead for maximum performance. No automatic implementation, use the Shortcut app on an iDevice to automate things and if no such automation can be used thatā€™s that then. You canā€™t really open a stream on Home when somebody is ringing your doorbell because first the Homebase gets the call then it must reroute to Homebridge and then the Home app will pick it up(too many redirects) Use the Eufy app for doorbell rings instead.

Dravegon commented 2 years ago

Steverae68&Karesake: Thatā€™s really unfortunate. Yeah i read up on about the RTSP function on Eufy forums and like everybody is asking for such stream and nobody is getting a concrete answer back from officials. Knowing(out of experience) Chinese companies they donā€™t give any on their older products because they already ahead on working on the 4th or 5th maybe 6th new product and left the ā€œoldā€ doorbell for what it is... Samsung anybody? lol

Thatā€™s why i always root for Apple products, decent support on what they put out. I guess you get what you pay for still stands out up until this day and age Even when they are among # 1 tech companies in the world they keep it real which is very important

github-actions[bot] commented 2 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

steverae68 commented 2 years ago

Happy to close this item as I have switched to a Netatmo HKSV doorbell so... no longer care.

Thank you for the amazing work you have achieved with this plugin, it has truely been one of the best hence the reason I have donated to this project.

I wish you every success in releasing your beta version with local storage support, I had very few issues with this but I am on a drive to replace all my non HK devices with native ones

Thanks again guys

lenoxys commented 2 years ago

please use the link above to update. I'm locking this topics since we're moving into a new repo. Sorry for the inconvenience.

lenoxys commented 2 years ago

@steverae68, we will plan to develop HKSV on the next release 1.2.

Thanks again for the tweak you provided it was helpful. See you around !