search

llaske / sugarizer

Sugarizer is a web implementation of the Sugar platform to run on any device or browser
https://sugarizer.org
Apache License 2.0
199 stars 424 forks source link

Recording video doesn't work on Windows #1203

Open VedantSharma11 opened 1 year ago

VedantSharma11 commented 1 year ago

Hello @llaske I wanted to bring to your attention an issue I found while using the Record Activity. Specifically, the VideoRecord feature does not seem to be working properly. When I click on the record button it starts recording the video for 5 seconds but when I try to preview it the video player does not work properly and it is not able to play the video, the photo and audio-record feature works properly. I have tested this on several different devices and it breaks only on windows devices, so I believe the issue is with the code itself.

I wanted to report this issue to you so that you could share your thoughts on it. if you find it as an issue I would like you to assign it to me, thank you.

https://user-images.githubusercontent.com/96445392/221550097-e92797df-ba18-40a7-81e2-e851713714a2.mp4

llaske commented 1 year ago

@VedantSharma11 which browser are you using? Is there any message in the console?

VedantSharma11 commented 1 year ago

@llaske i am not running it on the browser i am running it locally on my system and it runs as a system application, and the above issue exists there.

llaske commented 1 year ago

How did you install it? How do you run it?

VedantSharma11 commented 1 year ago

I forked and cloned the sugarizer repository on my local machine and then i installed the packages using "npm install" and to run it locally i used "npm start"

llaske commented 1 year ago

Okay run it with debug option and tell me if there is error message on the devtools console when you try to record a video.

 npm start -- --sdebug --window
VedantSharma11 commented 1 year ago

It gives some warnings on the devtools console. I am attaching the snippet for the same

Screenshot 2023-02-28 030359

llaske commented 1 year ago

Okay. Look like a security issue. Could you install Sugarizer using the Windows installer instead (see here) to check if it fix the issue?

VedantSharma11 commented 1 year ago

@llaske I installed sugarizer using the windows installer and i tried the record activity but the issue still persists

Vishalk91-4 commented 11 months ago

@llaske, is this issue open to work upon

llaske commented 11 months ago

@Vishalk91-4 sure.

llaske commented 1 month ago

Fixed in https://github.com/llaske/sugarizer/pull/1664