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

Unhandled Promise Rejection: TypeError: Argument 1 ('options') to RTCPeerConnection.createOffer must be a dictionary #1550

Closed
joshuajazleung opened this issue Nov 23, 2018 · 8 comments
Labels

Comments

@joshuajazleung
Copy link

@joshuajazleung joshuajazleung commented Nov 23, 2018

What version of WebTorrent?
v0.102.4

What operating system and Node.js version?
MacOS Majove, Nove v10.11.0

What browser and version? (if using WebTorrent in the browser)
Safari 12

What did you expect to happen?
I expect the web torrent client to run normally

What actually happened?
it doesn't run at all and displays the error message in console.

Not only on my app, https://instant.io/ has the same problem. If you open a "share link" from instant.io on a new safari tab, it doesn't run at all.

@jimmywarting

This comment has been minimized.

Copy link
Contributor

@jimmywarting jimmywarting commented Nov 23, 2018

I think this belongs more to the simple-peer repo

Basically RTCSessionDescription and RTCIceCandidate constructor are deprecated and should just pass an object instead of a instance

@joshuajazleung

This comment has been minimized.

Copy link
Author

@joshuajazleung joshuajazleung commented Nov 23, 2018

I think this belongs more to the simple-peer repo

Basically RTCSessionDescription and RTCIceCandidate constructor are deprecated and should just pass an object instead of a instance

But web torrent referenced the old simple-peer. The current one is v9.1.2 while webtorrent is referencing v9.0.0

@jimmywarting

This comment has been minimized.

Copy link
Contributor

@jimmywarting jimmywarting commented Nov 23, 2018

the ^ in "^9.0.0" means it will automatically get the latest minor patch so when u install webtorrent u will get v9.1.2. it's like 9.x.x

if it would be ~ ~9.0.0 then u get all latest bug patches 9.0.x

@joshuajazleung

This comment has been minimized.

Copy link
Author

@joshuajazleung joshuajazleung commented Nov 24, 2018

the ^ in "^9.0.0" means it will automatically get the latest minor patch so when u install webtorrent u will get v9.1.2. it's like 9.x.x

if it would be ~ ~9.0.0 then u get all latest bug patches 9.0.x

So that means we have to wait for simple-peer to update?

@jimmywarting

This comment has been minimized.

Copy link
Contributor

@jimmywarting jimmywarting commented Nov 24, 2018

So that means we have to wait for simple-peer to update?

yepp

@stale

This comment has been minimized.

Copy link

@stale stale bot commented Feb 22, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@froduit

This comment has been minimized.

Copy link

@froduit froduit commented Mar 20, 2019

still not fixed ?

@lock lock bot locked as resolved and limited conversation to collaborators Jun 18, 2019
@webtorrent webtorrent unlocked this conversation Aug 11, 2019
@feross

This comment has been minimized.

Copy link
Member

@feross feross commented Aug 11, 2019

@froduit simple-peer has been updated and this is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.