Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

NullReferenceException - LookupClient.ProcessResponseMessage #136

Closed
Silvenga opened this issue Oct 10, 2021 · 3 comments · Fixed by #146
Closed

NullReferenceException - LookupClient.ProcessResponseMessage #136

Silvenga opened this issue Oct 10, 2021 · 3 comments · Fixed by #146
Labels
Milestone

Comments

@Silvenga
Copy link

I haven't been able to reproduce this on demand, it only occurs under heavy load.

DnsClient.DnsResponseException: Query 43835 => pull-cmaf-f5.tiktokcdn.com IN A on 10.43.37.252:53 failed with an error.
---> System.NullReferenceException: Object reference not set to an instance of an object.
  at DnsClient.LookupClient.ProcessResponseMessage(LookupClientAudit audit, DnsRequestMessage request, DnsResponseMessage response, DnsQuerySettings settings, NameServer nameServer, DnsMessageHandleType handleType, Int32 serverCount, Boolean isLastServer, HandleError& handleError)
  at DnsClient.LookupClient.ResolveQueryAsync(IReadOnlyList`1 servers, DnsQuerySettings settings, DnsMessageHandler handler, DnsRequestMessage request, LookupClientAudit audit, CancellationToken cancellationToken)
  --- End of inner exception stack trace ---
  at DnsClient.LookupClient.ResolveQueryAsync(IReadOnlyList`1 servers, DnsQuerySettings settings, DnsMessageHandler handler, DnsRequestMessage request, LookupClientAudit audit, CancellationToken cancellationToken)
  at DnsClient.LookupClient.QueryInternalAsync(DnsQuestion question, DnsQuerySettings queryOptions, IReadOnlyCollection`1 servers, CancellationToken cancellationToken)

The DnsResponseException had:

Code: Unassigned
DnsError: 'Unknown Error'

10.43.37.252:53 is an unbound server acting in recursive mode with rate-limiting disabled.

@Silvenga
Copy link
Author

Silvenga commented Oct 10, 2021

Retrying does allow the request to succeed the second time most of the time, Trace logging doesn't have anything useful. Nothing obvious jumps out in the code. 🤔

@MichaCo
Copy link
Owner

MichaCo commented Oct 11, 2021

Hi, thanks for the report.

Yeah, it at least shouldn't throw a null reference exceptions. I'll take a look

@MichaCo MichaCo added this to the 1.6.0 milestone Jan 26, 2022
@MichaCo MichaCo added the bug label Jan 26, 2022
MichaCo added a commit that referenced this issue Jan 30, 2022
Removing UDP client pooling 
+ some bug fixes: #132, #145, #136, #133
@MichaCo
Copy link
Owner

MichaCo commented Jan 30, 2022

The occurrence of a NullReferenceException should be fixed in 1.6. Not 100% sure though so let me know if that still happens

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants