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

Enable WebRTC connections to be made to the Domain, i.e., add Web client support. #1469

Merged
merged 112 commits into from
Nov 26, 2021

Conversation

ctrlaltdavid
Copy link
Collaborator

Enables browser clients using the Web SDK to connect to the domain server and assignment clients via WebRTC.

image

WebRTC connections can be enabled/disabled in the server settings. An SSL certificate is required if running a public domain - the Web client needs to be served via HTTPS in order for Web audio to work, which means that the WebSocket used to establish the WebRTC connection with the domain server also needs to be SSL. (If running on localhost or local network, the Web client can be served via HTTP and Web audio will still work, and no SSL cert is required.)

ctrlaltdavid and others added 30 commits May 20, 2021 09:43
Conditionally include WebRTCSignalingServer.h
…annel

# Conflicts:
#	domain-server/src/DomainServer.h
Update webapp feature branch with current master
@ctrlaltdavid ctrlaltdavid added enhancement New feature or request packet version change will result in a protocol mismatch domain-server Issues related to the domain server in general needs testing (QA) The PR is ready for testing labels Nov 23, 2021
@digisomni digisomni added this to the 2022.1.0 Selene Release milestone Nov 23, 2021
@digisomni digisomni changed the title Enable WebRTC connections to be made to the domain, i.e., add Web client support. Enable WebRTC connections to be made to the Domain, i.e., add Web client support. Nov 23, 2021
Copy link
Member

@digisomni digisomni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OS Name Microsoft Windows 10 Pro
Version 10.0.19042 Build 19042

Works fine to me with basic local testing between native and web on a local server.

@ksuprynowicz
Copy link
Contributor

I tested it with Kalila on Linux server and both native and web clients, and everything, including chat and messages works perfectly.

@ksuprynowicz
Copy link
Contributor

Windows server functionality is also tested now and works correcly except for slightly choppy audio from web client.

@digisomni digisomni added QA Approved The PR has been tested successfully. merge right before snip Do not merge these changes until the very end. and removed needs testing (QA) The PR is ready for testing labels Nov 25, 2021
@digisomni digisomni merged commit aabcdea into master Nov 26, 2021
JulianGro added a commit to JulianGro/vircadia that referenced this pull request Dec 15, 2021
This reverts commit aabcdea, reversing
changes made to 57167cd.

# Conflicts:
#	libraries/networking/src/NodeList.h
@digisomni digisomni deleted the webapp branch November 21, 2022 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-server Issues related to the domain server in general enhancement New feature or request merge right before snip Do not merge these changes until the very end. packet version change will result in a protocol mismatch QA Approved The PR has been tested successfully.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants