Avoid changing the timeouts selected but the user

zmap / zdns

Fast DNS Lookup Library and CLI Tool

Apache License 2.0

891 stars 121 forks source link

Avoid changing the timeouts selected but the user #363

Closed phillip-stephens closed 2 months ago

phillip-stephens commented 3 months ago

Inside retryingLookup here, ZDNS is doubling the user-provided timeout for every retry that the user provides. While very good-intentioned, IMO ZDNS should provide an appropriate error to the user and leave it to them to increase the timeout.

phillip-stephens commented 3 months ago

I'm currently almost done with a refactor of ZDNS and will address this after.