Skip to content
Snippets Groups Projects
Commit 4f67b5f1 authored by Adrien Béraud's avatar Adrien Béraud
Browse files

sipcall: call media start/stop only after SDP negotiation is done

This patch makes also ICE ready non-blocking.

Refs #67233
Refs #67319

Change-Id: I0bc515078dd4276220430007845df040f27a5cc7
parent 33ad64f4
No related branches found
No related tags found
No related merge requests found
......@@ -276,10 +276,6 @@ void SIPCall::answer()
throw std::runtime_error("Could not send invite request answer (200 OK)");
}
if (iceTransport_->isStarted())
waitForIceNegotiation(DEFAULT_ICE_NEGO_TIMEOUT);
startAllMedia();
setConnectionState(CONNECTED);
setState(ACTIVE);
}
......@@ -665,9 +661,6 @@ void
SIPCall::onAnswered()
{
if (getConnectionState() != Call::CONNECTED) {
if (iceTransport_->isStarted())
waitForIceNegotiation(DEFAULT_ICE_NEGO_TIMEOUT);
startAllMedia();
setConnectionState(Call::CONNECTED);
setState(Call::ACTIVE);
Manager::instance().peerAnsweredCall(*this);
......@@ -818,16 +811,28 @@ SIPCall::stopAllMedia()
void
SIPCall::onMediaUpdate()
{
stopAllMedia();
openPortsUPnP();
// Handle possible ICE transport
if (!startIce())
RING_WARN("ICE not started");
if (getState() == ACTIVE) {
// TODO apply changes without restarting everything
RING_WARN("Restarting medias");
stopAllMedia();
if (startIce()) {
auto this_ = std::static_pointer_cast<SIPCall>(shared_from_this());
auto iceTimeout = std::chrono::steady_clock::now() + std::chrono::seconds {10};
Manager::instance().addTask([=] {
/* First step: wait for an ICE transport for SIP channel */
if (this_->iceTransport_->isFailed() or std::chrono::steady_clock::now() >= iceTimeout) {
RING_DBG("ice init failed (or timeout)");
this_->setConnectionState(Call::DISCONNECTED);
Manager::instance().callFailure(*this_); // signal client
this_->removeCall();
return false;
}
if (not this_->iceTransport_->isRunning())
return true;
startAllMedia();
return false;
});
} else {
RING_WARN("Starting medias without ICE");
startAllMedia();
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment