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

fix: video having no duration (with recordrtc dependency from GitHub) #490

Closed

Conversation

marob
Copy link
Contributor

@marob marob commented Feb 18, 2022

When recording the presentation, the resulting video is corrupted as the duration is not/incorrectly detected by video players.

This issue explains the problem and gives a solution: we have to wrap the blob in getSeekableBlob function.

An important thing to note is that getSeekableBlob function requires EBML as dependency.

I have developed several PR solving the problem and using different ways to import EBML as each of them have pros and cons.

This PR imports recordrtc dependency twice (second time from GitHub URL instead of from npm registry) in order to have the required EBML.js file (which is not packed in the npm registry version of recordrtc)
Cons: we declare the same dependency twice and we use GitHub dependency (not a good practice?).

See other PRs:

We're importing recordrtc dependency twice (second time from github URL instead of from npm registry) in order to have the required EBML.js file (which is not packed in the npm registry version)
@marob marob changed the title fix: video having no duration fix: video having no duration (with recordrtc dependency from GitHub) Feb 18, 2022
@antfu
Copy link
Member

antfu commented Sep 12, 2022

Thanks but I'd prefer to have this fixed upstream.

@antfu antfu closed this Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants