-
Notifications
You must be signed in to change notification settings - Fork 40
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 DOMException #25
Comments
@brulufa9 I'm getting this error too using Chrome 83.0.4103.44 64 bits. Did you find out how to bypass it? |
Sadly i did not. But if you do, please give me a heads up! |
Same |
Exact same error here :(
|
Same in Chrome 83 and Chrome Canary 85 :( |
I get the same error with Chrome 83 |
Any update regarding this error? I started getting it on chrome. |
I get the same error since today morning for Chrome 83.0.4103.61. |
same problem. |
What I did to bypass the problem temporarily is to remove opus media library and used MediaRecorder. |
but, we need convert to ogg :( |
so far I don't know how to fix it. this coincides with this change that was made. in encoderWorker.js case 'pushInputData':
I think the problem is here. |
I found the problem :) this.worker.postMessage({ remove final that worked |
@thiagoamarante, thanks a lot for your hint. I can confirm that after applying your changes and rebuilding the opus-media-recorder from the sources, everything also worked fine for me. I tested with the latest version of Chrome but also Firefox, Opera and Edge under Windows 10, as well as with the latest versions of Chrome and Firefox under Android - and I can tell you that it works like a charme ;-). Thus @thiagoamarante do you want to hand in a pull request, so that other people later on hopefully can download the fixed opus-media-recorder via npm? UpdateIn the meantime, my team and me also already tested the Apple world and I can confirm that using the latest Safari under Mac and iOS also works with the fix applied. |
I tried compile but have error :( @thiagoamarante, @maximilianlorent , can u send compiled files here !? [] s! |
@borigoto, find attached all files that were generated during the build. As I also had many problems during the build let me summarize what I did to make the build process work properly:
Hope these hints might help a bit. Nevertheless, @kbumsik do you have the time to update this Repo here and provide a new NPM package based on the solution of @thiagoamarante ? If you don't have much time at the moment, is it a feasible option for you if I hand in a pull request when I find the time and you just update the NPM? |
Hi guys, I haven't take care of this library since I started a new Emscripten project. I was a little bit surprised by that people started using it open this kinds of issue since then. (The download count gets 5 times more since the Coronavirus break. Maybe this package got some attention because this library somewhat related to WFH?) Alright, it's time to do something here. I will look into this issue and try to resolve the issue until this weekend. Even more, I will set up a CI test & build for more reliable maintenance. The CI setup may take a couple of weeks. |
@maximilianlorent , Hi! Many thanks!! Tested and all work ok!!! :-) Yes, i change sources from gitmodules, and, read your "instructions" i think my problem is version of Emscripten. [] s |
Attached is the corrected file |
|
Hi guys, |
@kbumsik Still seeing the issue after updating to 0.8.0:
|
Any ideas on why i'm getting this error. I called
MediaRecorder.start(1000)
, using the timeslice and my onDataAvalible function is not being called. The code works without Opus.The text was updated successfully, but these errors were encountered: