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.

Sign up for GitHub

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

Jump to bottom

*PeerConnection has no field or method SCTP #2099

Closed
anacrolix opened this issue Jan 25, 2022 · 1 comment · Fixed by #2120
Closed

*PeerConnection has no field or method SCTP #2099

anacrolix opened this issue Jan 25, 2022 · 1 comment · Fixed by #2120

Comments

@anacrolix
Copy link

anacrolix commented Jan 25, 2022 •
edited
Loading

From https://github.com/anacrolix/torrent/blob/c47e6b1f605143e1f0f8d2172c2b5dfadd8fff71/webtorrent/tracker-client.go#L71, CI unexpectedly reports https://github.com/anacrolix/torrent/runs/4905753701?check_suite_focus=true

Run GOOS=js GOARCH=wasm go build . ./storage ./tracker/...
# github.com/anacrolix/torrent/webtorrent
Error: webtorrent/tracker-client.go:71:26: me.peerConnection.SCTP undefined (type *wrappedPeerConnection has no field or method SCTP)
Error: Process completed with exit code 2.

It looks like this method is not implemented for build tag js in package webrtc.
@Sean-Der Sean-Der added this to the 3.2.0 milestone Feb 5, 2022
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
7f60c1f 
Not available in wrtc

Resolves #2099
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
74eef92 
Not available in wrtc

Resolves #2099
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
1be193e 
Not available in wrtc

Resolves #2099
@Sean-Der Sean-Der mentioned this issue Feb 8, 2022
Merged
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
708ace6 
Only available in the browser, not in wrtc yet.

Resolves #2099
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
496bb71 
Only available in the browser, not in wrtc yet.

Resolves #2099
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
9f8d6d6 
Only available in the browser, not in wrtc yet.

Resolves #2099
Sean-Der added a commit that referenced this issue Feb 8, 2022
@Sean-Der
Add DTLS, ICE and SCTP Transport to WASM
94262c1 
Only available in the browser, not in wrtc yet.

Resolves #2099
@anacrolix
Copy link
Author

Thanks. Fixes downstream anacrolix/torrent@aa3569b.

boks1971 pushed a commit that referenced this issue Feb 9, 2022
@Sean-Der @boks1971
Add DTLS, ICE and SCTP Transport to WASM
5a5c1ce 
Only available in the browser, not in wrtc yet.

Resolves #2099
daonb pushed a commit to daonb/webrtc that referenced this issue Nov 20, 2022
@Sean-Der @daonb
Add DTLS, ICE and SCTP Transport to WASM
f6c4bf7 
Only available in the browser, not in wrtc yet.

Resolves pion#2099
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add DTLS, ICE and SCTP Transport to WASM pion/webrtc
2 participants
@anacrolix @Sean-Der