-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Move offer creation logic out of Erizo #1111
Move offer creation logic out of Erizo #1111
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍 Just remember to put the publicIP conversion back in there, see comment.
@@ -105,8 +105,8 @@ exports.ErizoJSController = function (threadPool, ioThreadPool) { | |||
|
|||
case CONN_SDP: | |||
case CONN_GATHERED: | |||
mess = mess.replace(that.privateRegexp, that.publicIP); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The conversion to set the proper publicIp in the candidate is missing here
return value; | ||
} | ||
|
||
function addSsrc(sources, ssrc, sdp, media) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 A big improvement over the old implementation
Description
It moves the logic to create SDP offer from Erizo to ErizoJS.
Changes in Client or Server public APIs
Not needed.
[] It includes documentation for these changes in
/doc
.