Skip to content

Commit

Permalink
Lower the severity of some messages
Browse files Browse the repository at this point in the history
When authoritative servers are dead or invalid, upstream resolvers
can reply after the proxy timeout is reached. This is totally
expected and should not be reported as something wrong to the user.
  • Loading branch information
jedisct1 committed Jan 5, 2015
1 parent 1cf3aa2 commit baf3146
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion tcp_request.c
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ timeout_timer_cb(evutil_socket_t timeout_timer_handle, short ev_flags,

(void)ev_flags;
(void)timeout_timer_handle;
logger(LOG_WARNING, "resolver timeout (TCP)");
logger(LOG_DEBUG, "resolver timeout (TCP)");
tcp_request_kill(tcp_request);
}

Expand Down
4 changes: 2 additions & 2 deletions udp_request.c
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ timeout_timer_cb(evutil_socket_t timeout_timer_handle, short ev_flags,

(void)ev_flags;
(void)timeout_timer_handle;
logger(LOG_WARNING, "resolver timeout (UDP)");
logger(LOG_DEBUG, "resolver timeout (UDP)");
udp_request_kill(udp_request);
}

Expand Down Expand Up @@ -461,7 +461,7 @@ resolver_to_proxy_cb(evutil_socket_t proxy_resolver_handle, short ev_flags,
}
udp_request = lookup_request(c, id, hash);
if (udp_request == NULL) {
logger(LOG_ERR, "Received a reply that doesn't match any active query");
logger(LOG_DEBUG, "Received a reply that doesn't match any active query");
return;
}
size_t max_reply_size = DNS_MAX_PACKET_SIZE_UDP;
Expand Down

0 comments on commit baf3146

Please sign in to comment.