Browse Source

Moved verbose logging when starting transport

Paul-Louis Ageneau 4 years ago
parent
commit
25b3e13d27
1 changed files with 6 additions and 3 deletions
  1. 6 3
      src/peerconnection.cpp

+ 6 - 3
src/peerconnection.cpp

@@ -396,11 +396,12 @@ void PeerConnection::onTrack(std::function<void(std::shared_ptr<Track>)> callbac
 }
 }
 
 
 shared_ptr<IceTransport> PeerConnection::initIceTransport() {
 shared_ptr<IceTransport> PeerConnection::initIceTransport() {
-	PLOG_VERBOSE << "Starting ICE transport";
 	try {
 	try {
 		if (auto transport = std::atomic_load(&mIceTransport))
 		if (auto transport = std::atomic_load(&mIceTransport))
 			return transport;
 			return transport;
 
 
+		PLOG_VERBOSE << "Starting ICE transport";
+
 		auto transport = std::make_shared<IceTransport>(
 		auto transport = std::make_shared<IceTransport>(
 		    mConfig, weak_bind(&PeerConnection::processLocalCandidate, this, _1),
 		    mConfig, weak_bind(&PeerConnection::processLocalCandidate, this, _1),
 		    [this, weak_this = weak_from_this()](IceTransport::State state) {
 		    [this, weak_this = weak_from_this()](IceTransport::State state) {
@@ -459,11 +460,12 @@ shared_ptr<IceTransport> PeerConnection::initIceTransport() {
 }
 }
 
 
 shared_ptr<DtlsTransport> PeerConnection::initDtlsTransport() {
 shared_ptr<DtlsTransport> PeerConnection::initDtlsTransport() {
-	PLOG_VERBOSE << "Starting DTLS transport";
 	try {
 	try {
 		if (auto transport = std::atomic_load(&mDtlsTransport))
 		if (auto transport = std::atomic_load(&mDtlsTransport))
 			return transport;
 			return transport;
 
 
+		PLOG_VERBOSE << "Starting DTLS transport";
+
 		auto certificate = mCertificate.get();
 		auto certificate = mCertificate.get();
 		auto lower = std::atomic_load(&mIceTransport);
 		auto lower = std::atomic_load(&mIceTransport);
 		auto verifierCallback = weak_bind(&PeerConnection::checkFingerprint, this, _1);
 		auto verifierCallback = weak_bind(&PeerConnection::checkFingerprint, this, _1);
@@ -530,11 +532,12 @@ shared_ptr<DtlsTransport> PeerConnection::initDtlsTransport() {
 }
 }
 
 
 shared_ptr<SctpTransport> PeerConnection::initSctpTransport() {
 shared_ptr<SctpTransport> PeerConnection::initSctpTransport() {
-	PLOG_VERBOSE << "Starting SCTP transport";
 	try {
 	try {
 		if (auto transport = std::atomic_load(&mSctpTransport))
 		if (auto transport = std::atomic_load(&mSctpTransport))
 			return transport;
 			return transport;
 
 
+		PLOG_VERBOSE << "Starting SCTP transport";
+
 		auto remote = remoteDescription();
 		auto remote = remoteDescription();
 		if (!remote || !remote->application())
 		if (!remote || !remote->application())
 			throw std::logic_error("Starting SCTP transport without application description");
 			throw std::logic_error("Starting SCTP transport without application description");