-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Invalid candidate format
When adding an ice candidate generated from browser WebRTC
#128
Comments
UPDATE: I switched to using SIP Sorcery for WebRTC communication and got it to work, check it out here. It still would be nice if the GDNative plugin worked especially since it seams so close to working. Maybe I'm doing something wrong somewhere? |
Would be great if you could post the candidate data that is failing so we can better understand if it's a problem with the Godot wrapper or libdatachannel itself (in which case we should report it upstream). |
Of course! The client sends 3 candidates in total + an empty one which is discarded and not sent:
Here are the complete logs: Sever Log
Client Log
|
Godot version
v4.2.1.rc1.official [daeb1c729]
Plugin version
1.0.3
System information
Fedora 39
Issue description
When trying to set up a WebRTC connect between Godot and the browser the browser successfully connects but Godot fails when receiving ice candidates from the browser (
Invalid candidate format
). According the libdatachannel it should be compatible with the browser. There also datachannel-wasm for the browser which might offer better compatibility but I have not done any testing because it would require setting up WebAssembly.Why am I doing all this? Although WebSockets exists the only way to send UDP (or UDP like) packets in a browser is with WebRTC or WebTransport. WebTransport is not supported in Safari yet, a deal breaker for smartphones leaving the only other option being WebRTC.
Steps to reproduce
See the attached example project.
Minimal reproduction project
Godot Remote - An in-development addon to provide functionality for using a smartphone as a remote controller, sort of like jackbox.tv.
The text was updated successfully, but these errors were encountered: