ApiListener: Fix misleading connection log messages

fixes #8493
This commit is contained in:
Michael Friedrich 2015-02-20 15:11:44 +01:00
parent e2815de8a6
commit ba77d7b35f
1 changed files with 5 additions and 5 deletions

View File

@ -380,7 +380,7 @@ void ApiListener::ApiTimerHandler(void)
/* only connect to endpoints in a) the same zone b) our parent zone c) immediate child zones */
if (my_zone != zone && my_zone != zone->GetParent() && zone != my_zone->GetParent()) {
Log(LogDebug, "ApiListener")
<< "Not connecting to Endpoint '" << zone->GetName() << "' because it's not in the same zone, a parent or a child zone.";
<< "Not connecting to Zone '" << zone->GetName() << "' because it's not in the same zone, a parent or a child zone.";
continue;
}
@ -396,7 +396,7 @@ void ApiListener::ApiTimerHandler(void)
/* don't connect to an endpoint if we already have a connection to the zone */
if (connected) {
Log(LogDebug, "ApiListener")
<< "Not connecting to Endpoint '" << zone->GetName() << "' because we're already connected to it.";
<< "Not connecting to Zone '" << zone->GetName() << "' because we're already connected to it.";
continue;
}
@ -404,21 +404,21 @@ void ApiListener::ApiTimerHandler(void)
/* don't connect to ourselves */
if (endpoint->GetName() == GetIdentity()) {
Log(LogDebug, "ApiListener")
<< "Not connecting to Endpoint '" << zone->GetName() << "' because that's us.";
<< "Not connecting to Endpoint '" << endpoint->GetName() << "' because that's us.";
continue;
}
/* don't try to connect to endpoints which don't have a host and port */
if (endpoint->GetHost().IsEmpty() || endpoint->GetPort().IsEmpty()) {
Log(LogDebug, "ApiListener")
<< "Not connecting to Endpoint '" << zone->GetName() << "' because the host/port attributes are missing.";
<< "Not connecting to Endpoint '" << endpoint->GetName() << "' because the host/port attributes are missing.";
continue;
}
/* don't try to connect if there's already a connection attempt */
if (endpoint->GetConnecting()) {
Log(LogDebug, "ApiListener")
<< "Not connecting to Endpoint '" << zone->GetName() << "' because we're already trying to connect to it.";
<< "Not connecting to Endpoint '" << endpoint->GetName() << "' because we're already trying to connect to it.";
continue;
}