Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1398)

Unified Diff: webrtc/p2p/base/turnport.cc

Issue 1493863002: Add logs when stun or turn host lookup is completed. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« webrtc/p2p/base/stunport.cc ('K') | « webrtc/p2p/base/stunport.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/p2p/base/turnport.cc
diff --git a/webrtc/p2p/base/turnport.cc b/webrtc/p2p/base/turnport.cc
index 20c2b6002f78db2ab505fc3668f5adc5cea9bcb3..e75b51793415ee72c3c839fef9de8d715d66be92 100644
--- a/webrtc/p2p/base/turnport.cc
+++ b/webrtc/p2p/base/turnport.cc
@@ -619,6 +619,8 @@ void TurnPort::ResolveTurnAddress(const rtc::SocketAddress& address) {
void TurnPort::OnResolveResult(rtc::AsyncResolverInterface* resolver) {
ASSERT(resolver == resolver_);
+ LOG_J(LS_INFO, this) << "TURN host lookup result " << resolver_->GetError();
+
// If DNS resolve is failed when trying to connect to the server using TCP,
// one of the reason could be due to DNS queries blocked by firewall.
// In such cases we will try to connect to the server with hostname, assuming
@@ -635,8 +637,6 @@ void TurnPort::OnResolveResult(rtc::AsyncResolverInterface* resolver) {
rtc::SocketAddress resolved_address = server_address_.address;
if (resolver_->GetError() != 0 ||
!resolver_->GetResolvedAddress(ip().family(), &resolved_address)) {
- LOG_J(LS_WARNING, this) << "TURN host lookup received error "
- << resolver_->GetError();
error_ = resolver_->GetError();
OnAllocateError();
return;
« webrtc/p2p/base/stunport.cc ('K') | « webrtc/p2p/base/stunport.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698