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

uncaught exception after stop_stream in .T8020 (Battery doorbell 1080p) #98

Closed
pablovalen opened this issue Mar 16, 2021 · 5 comments
Closed
Assignees
Labels
bug Something isn't working fixed. waiting for feedback Issue should be fixed. Waiting on feedback

Comments

@pablovalen
Copy link

pablovalen commented Mar 16, 2021

Describe the bug
A clear and concise description of what the bug is:
When I try to stop stream, I get:
uncaught exception: ffmpeg exited with code 255
And then:
instance system.adapter.eufy-security.0 terminated with code 6 (UNCAUGHT_EXCEPTION)

After it, I need to manual restart instance to be able to stream again.

To Reproduce
Steps to reproduce the behavior:
1.- I click on start_stream.
2.- I can access to live stream.
3.- I click on stop_stream.
4.- Log error in silly mode:

`

host.bee74d33d7fb 2021-03-16 08:42:14.203 info Restart adapter system.adapter.eufy-security.0 because enabled
host.bee74d33d7fb 2021-03-16 08:42:14.201 error instance system.adapter.eufy-security.0 terminated with code 6 (UNCAUGHT_EXCEPTION)
eufy-security.0 2021-03-16 08:42:13.744 info (27671) Disconnected from station T8020P1020341B19.
eufy-security.0 2021-03-16 08:42:13.675 debug (27671) P2PClientProtocol.onClose():
eufy-security.0 2021-03-16 08:42:13.675 warn (27671) Terminated (UNCAUGHT_EXCEPTION): Without reason
eufy-security.0 2021-03-16 08:42:13.675 info (27671) terminating
eufy-security.0 2021-03-16 08:42:13.675 info (27671) Disconnect from station T8020P1020341B19.
eufy-security.0 2021-03-16 08:42:13.672 silly (27671) States system redis pmessage io.messagebox.system.adapter.eufy-security.0/io.messagebox.system.adapter.eufy-security.0:{"command":"addNotification","message":{"result":"ok"},"from":"system.hos
eufy-security.0 2021-03-16 08:42:13.668 error (27671) ffmpeg exited with code 255: frame= 103 fps=9.8 q=-1.0 Lsize=N/A time=00:00:13.19 bitrate=N/A speed=1.25x video:959kB audio:35kB subtitle:0kB other streams:0kB global headers:0kB muxing o
eufy-security.0 2021-03-16 08:42:13.668 error (27671) Error: ffmpeg exited with code 255: frame= 103 fps=9.8 q=-1.0 Lsize=N/A time=00:00:13.19 bitrate=N/A speed=1.25x video:959kB audio:35kB subtitle:0kB other streams:0kB global headers:0kB m
eufy-security.0 2021-03-16 08:42:13.667 error (27671) uncaught exception: ffmpeg exited with code 255: frame= 103 fps=9.8 q=-1.0 Lsize=N/A time=00:00:13.19 bitrate=N/A speed=1.25x video:959kB audio:35kB subtitle:0kB other streams:0kB global
eufy-security.0 2021-03-16 08:42:12.690 debug (27671) state eufy-security.0.T8020P1020341B19.cameras.T8222P002034304E.livestream deleted
eufy-security.0 2021-03-16 08:42:12.689 silly (27671) States user redis pmessage eufy-security.0.T8020P1020---B--.*/eufy-security.0.T8020P1020341B19.cameras.T8222P0020---B--.livestream:null
eufy-security.0 2021-03-16 08:42:12.677 silly (27671) onStopLivestream(): station: T8020P1020---B-- device: T8222P002034304E
eufy-security.0 2021-03-16 08:42:12.675 info (27671) Livestream of camera T8222P0020---B-- stopped.
eufy-security.0 2021-03-16 08:42:12.674 debug (27671) BatteryDoorbellCamera.stopStream(): Response: {"code":0,"msg":"Succeed."}

`

Expected behavior
I understand what should happen is the stream stops and, if there is an error in ffmpeg, should not impact the instance in an uncaught exception.

After that, if no manual restart is done, its not possible to start_stream again before doing a manual restart of the instance.

Versions:

  • Adapter version: 0.4.2
  • JS-Controller version: dont know.
  • Node version: v12.21.0
  • Operating system: Debian
@bropat bropat self-assigned this Mar 16, 2021
@bropat bropat added the bug Something isn't working label Mar 16, 2021
@pablovalen
Copy link
Author

Hi, if you need any help from my side, I can try to help you :)

@bropat
Copy link
Owner

bropat commented Mar 17, 2021

Hi @PabloVal-coder,

thank you, I will let you know.

@bropat bropat added the work in progress Working on it label Mar 17, 2021
@bropat
Copy link
Owner

bropat commented Mar 30, 2021

I found the following closed issue for the lib I am using:

fluent-ffmpeg/node-fluent-ffmpeg#644

Will check and implement the solution. I will let you know.

@pablovalen
Copy link
Author

Thats great 👍

bropat added a commit that referenced this issue Aug 13, 2021
…tates will be dropped and recreated (Note: some states where renamed)

Supports Admin 5
Added new adapter setting "Accept incoming invitations" to automatically accept device invitations
Added new adapter setting "Alarm sound duration (sec)" used for triggering alarm sound on supported devices/stations (#76)
Added enable/disable led setting for camera 1 products
Added motion detection sensitivity setting for camera 1 products and wired doorbell
Added motion detection type setting for camera 1 products
Added motion audio recording setting for camera 1 products and wired doorbell
Added ringtone volume setting for wired doorbell
Added enable/disable indoor chime setting for wired doorbell (#100)
Added notification ring setting for wired doorbell
Added notification motion setting for wired doorbell
Added video streaming quality setting for wired doorbell
Added video recording quality setting for wired doorbell
Added video HDR setting for wired doorbell
Added video distortion correction setting for wired doorbell
Added video ring recording setting for wired doorbell
Added notification type setting for camera 1 products, solo cameras and wired doorbell
Added chirp volume setting for entry sensor
Added chirp tone setting for entry sensor
Added pan an tilt functionality to supported indoor cameras (#129)
Added error detection if stopping or starting stream that isn't running or already running
Added new setting "acceptInvitations" to "EufySecurity" to accept invitations automatically
Added floodlight camera light switch (#133)
Added motion detection sensitivity for indoor cameras, solo cameras, floodlight cameras, camera 2 products and battery doorbells (#133)
Added motion detection type for indoor cameras, solo cameras, floodlight cameras, camera 2 products and battery doorbells (#133)
Added motion tracking for indoor camera pan & tilt cameras
Added video stream quality setting for indoor cameras, solo cameras, floodlight cameras and battery doorbell (#133)
Added video recording quality setting for indoor cameras
Added WDR setting for battery doorbells
Added microphone mute setting for indoor cameras, solo cameras, floodlight cameras, camera 2 products and battery doorbells (#133)
Added audio recording setting for indoor cameras, solo cameras, floodlight cameras, camera 2 products and battery doorbells (#133)
Added enable/disable speaker setting for indoor cameras, solo cameras, floodlight cameras, camera 2 products (#133)
Added speaker volume setting for indoor cameras, solo cameras, floodlight cameras, camera 2 products and battery doorbells (#133)
Added power source setting for camera 2 products cameras, eufy cameras and eufy E cameras
Added power working mode setting for solo cameras, camera 2 products, battery doorbells, eufy cameras and eufy E cameras
Added power custom working mode recording clip length setting for solo cameras, floodlight cameras, camera 2 products, battery doorbells, eufy cameras and eufy E cameras (#133)
Added power custom working mode recording retrigger interval setting for solo cameras, floodlight cameras, camera 2 products, battery doorbells, eufy cameras and eufy E cameras (#133)
Added power custom working mode recording ends if motion stops setting for solo cameras, floodlight cameras, camera 2 products, battery doorbells, eufy cameras and eufy E cameras (#133)
Added video streaming quality setting for indoor cameras, solo cameras, floodlight cameras, 2c pro cameras and battery doorbells (#133)
Added video recording quality setting for indoor 2k cameras and 2c pro cameras
Added motion detection sensitivity setting for indoor cameras, floodlight cameras and camera 2 products (#133)
Added enable/disable motion tracking setting for indoor pan & tilt cameras
Added motion detection type setting for indoor cameras, solo cameras, floodlight cameras, camera 2 products and battery doorbells (#133)
Added enable/disable WDR setting for battery doorbells
Added ringtone volume setting for battery doorbells
Added enable/disable chime indoor setting for battery doorbells (#100)
Added enable/disable chime homebase setting for battery doorbells (#100)
Added chime homebase ringtone volume setting for battery doorbells
Added chime homebase ringtone type setting for battery doorbells
Added notification type setting for solo cameras, floodlight cameras, camera 2 products, battery doorbells, eufy cameras and eufy E cameras (#133)
Added enable/disable person notification setting for indoor cameras
Added enable/disable pet notification setting for indoor cameras
Added enable/disable all other motion notification setting for indoor cameras
Added enable/disable all sound notification setting for indoor cameras
Added enable/disable crying notification setting for indoor cameras
Added enable/disable motion notification setting for battery doorbells
Added enable/disable ring notification setting for battery doorbells
Added trigger alarm sound for camera 2 products, indoor cameras, solo cameras (incl. new) and floodlight cameras (#76)
Added reset alarm sound for camera 2 products, indoor cameras, solo cameras (incl. new) and floodlight cameras (#76)
Added trigger alarm sound for homebase 1+2 (#76)
Added reset alarm sound for homebase 1+2 (#76)
Added alarm tone setting for homebase 1+2
Added alarm volume setting for homebase 1+2
Added prompt volume setting for homebase 1+2
Added time format setting for homebase 1+2
Added enable/disable switch mode app notification setting for homebase 1+2
Added enable/disable switch mode geofence notification setting for homebase 1+2
Added enable/disable switch mode schedule notification setting for homebase 1+2
Added enable/disable switch mode keypad notification setting for homebase 1+2
Added enable/disable start alarm delay notification setting for homebase 1+2
Added new floodlight, solo and outdoor cameras (untested!)
Added brightness light setting for 2c/2c pro cameras, new solo cameras and new outdoor cameras
Added enable/disable light setting for 2c/2c pro cameras, new solo cameras and new outdoor cameras
Added battery charging state for keypad devices
Added wifi rssi state for keypad devices
Added nightvision setting for devices supporting the "light" nightvision mode
Added enable disable "switch mode with access code" for station with registered keypad
Added enable disable "auto end alarm" for station with registered keypad
Added enable disable "turn off alarm with button" for station with registered keypad
Fixed issue #98
Fixed issue #140
Fixed issue #146
Fixed issue #117
Many small bugfixes
Updated versions of the package dependencies
@bropat bropat added fixed. waiting for feedback Issue should be fixed. Waiting on feedback and removed work in progress Working on it labels Aug 13, 2021
@bropat
Copy link
Owner

bropat commented Aug 13, 2021

@PabloVal-coder Please try version 0.6.0 and let me know.

@bropat bropat closed this as completed Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed. waiting for feedback Issue should be fixed. Waiting on feedback
Projects
None yet
Development

No branches or pull requests

2 participants