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

Support custom ICE ufrag and pwd fields via config #249

Conversation

achingbrain
Copy link

This is to enable setting a pre-shared ufrag/pwd as part of implementing libp2p WebRTC-Direct.

Refs: paullouisageneau/libdatachannel#1166

This is to enable setting a pre-shared ufrag/pwd as part of implementing
libp2p WebRTC-Direct.
@achingbrain
Copy link
Author

Apologies in advance if I've made some basic errors - C is not my forte though I've tried to add a test..

achingbrain added a commit to achingbrain/libdatachannel that referenced this pull request Jun 5, 2024
Adds two new optional config keys - `iceUfrag` and `icePwd` which
are passed to libjuice.

Depends on paullouisageneau/libjuice#249
achingbrain added a commit to achingbrain/libdatachannel that referenced this pull request Jun 5, 2024
Adds two new optional config keys - `iceUfrag` and `icePwd` which
are passed to libjuice.

Depends on: paullouisageneau/libjuice#249

Refs: paullouisageneau#1166
achingbrain added a commit to achingbrain/libdatachannel that referenced this pull request Jun 5, 2024
Adds two new optional config keys - `iceUfrag` and `icePwd` which
are passed to libjuice.

Depends on: paullouisageneau/libjuice#249

Refs: paullouisageneau#1166
@paullouisageneau
Copy link
Owner

If I'm not mistaken, this was already added by @xicilion in #243

@achingbrain
Copy link
Author

Very true! Not enough coffee, sorry..

@achingbrain achingbrain closed this Jun 5, 2024
@achingbrain achingbrain deleted the feat/allow-overriding-ufrag-and-pwd branch June 5, 2024 15:12
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