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

The stop recording button is making this error: #127

Closed
kanon4us opened this issue Jul 24, 2021 · 11 comments · Fixed by #133
Closed

The stop recording button is making this error: #127

kanon4us opened this issue Jul 24, 2021 · 11 comments · Fixed by #133
Labels

Comments

@kanon4us
Copy link

vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1 Uncaught (in promise) Error: No schema entry found for unknown
at EBMLEncoder.writeTag (vendors
main.d3cc3ca2e84b8b582969.bundle.js:1)
at EBMLEncoder.encodeChunk (vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1)
at vendors
main.d3cc3ca2e84b8b582969.bundle.js:1
at Array.reduce ()
at EBMLEncoder.encode (vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1)
at vendors
main.d3cc3ca2e84b8b582969.bundle.js:1
at Array.reduce ()
at encodedSizeOfEbml (vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1)
at Object.makeMetadataSeekable (vendors
main.d3cc3ca2e84b8b582969.bundle.js:1)
at main.d3cc3ca2e84b8b582969.bundle.js:1
EBMLEncoder.writeTag @ vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1
EBMLEncoder.encodeChunk @ vendors
main.d3cc3ca2e84b8b582969.bundle.js:1
(anonymous) @ vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1
EBMLEncoder.encode @ vendors
main.d3cc3ca2e84b8b582969.bundle.js:1
(anonymous) @ vendorsmain.d3cc3ca2e84b8b582969.bundle.js:1
encodedSizeOfEbml @ vendors
main.d3cc3ca2e84b8b582969.bundle.js:1
makeMetadataSeekable @ vendors~main.d3cc3ca2e84b8b582969.bundle.js:1
(anonymous) @ main.d3cc3ca2e84b8b582969.bundle.js:1
Promise.then (async)
(anonymous) @ main.d3cc3ca2e84b8b582969.bundle.js:1

@kanon4us
Copy link
Author

This is happening in Google Chrome Desktop at https://react-video-recorder.vercel.app/?path=/story/videorecorder--with-isflipped-false

@codyromano
Copy link

I'm experiencing the same issue.

@kanon4us
Copy link
Author

kanon4us commented Jul 25, 2021

What I can tell, the video schema object isn’t being set. In firefox, it is set to this...
schema
So if there is a way to manually set it, it might resolve the issue...
However, I don't think it's possible to set it manually due to it being one part of the data stream...
Screen Shot 2021-07-24 at 3 29 38 PM
Looking at the code just before the function that’s throwing the error, there are 72 array objects that each contain different schema types. Each of these are autogenerated as part of the video record process...
Screen Shot 2021-07-24 at 3 34 23 PM
That shows wherein the generated metadata the issue block is.
In this previous screenshot, you can see how the rest of the growing entries are “SimpleBlock”, which represent a chunk of the actual video...
I ran the video code in a Chromium browser which is a version or two behind Chrome, so it’s definitely a browser update that caused the issue.
Chromium uses the same rendering engine as Chrome, the only difference is that it has all the google tracking stripped from the browser.
I’m having no luck figuring out a workaround, so I think using another library is the only option at this time...
Here is another that I came across that supports react and works in Chrome...
https://collab-project.github.io/videojs-record/#/frameworks/react

@jackguoAtJogg
Copy link

I hope this could get fixed soon since this library comes better UI than https://collab-project.github.io/videojs-record/#/frameworks/react.

@asanjeevrao
Copy link

Facing the same issue

@jmazier-j2d
Copy link

Same issue here.. Any news ? I'm also trying to figure out a workaround, I'll let you know.

@jackguoAtJogg
Copy link

I think I found the issue, is due to this library https://github.com/legokichi/ts-ebml that's not working properly.

reference issue: legokichi/ts-ebml#33

@DevLukeHere
Copy link

Yup, facing the same issue here. I think it's broken for users using a combination of MacOS + chromium browsers (i.e. Google Chrome or Microsoft Edge).

@alexeymetelkinpro
Copy link

Hey there! Here is a workaround for that issue for now - #130;
Works for me!

I also forked that fork fix to make it usable through npm: here is my fork - https://github.com/alexeymetelkinpro/react-video-recorder.
To get the fix, you gotta replace the original react-video-recorder line with this - "react-video-recorder": "alexeymetelkinpro/react-video-recorder". Or "react-video-recorder": "alexeymetelkinpro/react-video-recorder#c4d02177de1f9f5e5f4db8bb242b195d1fed0899" to make sure you won't get further updates from my repository, though I have no plans to ever make any changes there.

@fbaiodias
Copy link
Owner

🎉 This issue has been resolved in version 3.19.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@codyromano
Copy link

@fbaiodias Thanks for your work on this! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants