bdgranger commented on issue #8417:
URL: https://github.com/apache/trafficserver/issues/8417#issuecomment-942655633
Based on our analysis and Susan's comment, we propose the following fix and
have put it into test:
The proposed fix would be to change the following in
HostDBContinuation::dnsEvent() in iocore/hostdb/HostDB.cc from:
```
if (event != DNS_EVENT_LOOKUP) {
// This was an event_interval or an event_immediate
// Either we timed out, or remove_trigger_pending gave up on us
if (!action.continuation) {
// give up on insert, it has been too long
hostDB.pending_dns_for_hash(hash.hash).remove(this);
hostdb_cont_free(this);
return EVENT_DONE;
}
```
to:
```
if (event != DNS_EVENT_LOOKUP) {
// This was an event_interval or an event_immediate
// Either we timed out, or remove_trigger_pending gave up on us
if (!action.continuation) {
// give up on insert, it has been too long
hostDB.pending_dns_for_hash(hash.hash).remove(this);
if (event == EVENT_INTERVAL) {
remove_trigger_pending_dns();
}
hostdb_cont_free(this);
return EVENT_DONE;
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
