DNS resolver can run for longer than the given timeout

Aperta

#101 aperta il 17/3/2023 4 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 4/5
Tempo stimato: 3-5 giorni
Idoneità per principianti: 38/100

Tipo di issue: Bug
Chiarezza: Abbastanza chiara
Stato di attività: Ferma
Stack tecnologico: python
Ambito: networking

Direzione di ricerca

Inizia in deliverability.py, nella chiamata dns_resolver.resolve(domain, "MX"), e traccia il modo in cui il timeout configurato viene passato quando check_availability è abilitato. Riproduci o analizza il ciclo del resolver e il comportamento di sleep descritti nell’issue. Il lavoro è completato quando la risoluzione DNS non prosegue oltre il timeout specificato, preservando il comportamento esistente di deliverability.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

It's very hard to reproduce, we found the problem because we have a 60s timeout for our web server that we kept busting, even with email-validator's default 15s timeout.

Disabling check_availability solved our problem.

The problem

The problem lies within:

response = dns_resolver.resolve(domain, "MX")

( https://github.com/JoshData/python-email-validator/blob/main/email_validator/deliverability.py#L40 )

While debugging, what I can see is that the resolver goes through a loop, and only checks the timeout between calls. It also does some time.sleep()
That means that if a call or a sleep is longer than the timeout, it doesn't get interrupted and can thus run for longer

The solution

Using signal, we could interrupt the process. See https://stackoverflow.com/a/494273

This package does it: https://github.com/pnpnpn/timeout-decorator

I'll try implementing the stackoverflow suggestion on my end and write back with news

Lingua principale: Python
Stelle: 1.4k
Fork: 140
Metriche di merge delle PR: Nessuna PR unita negli ultimi 30g

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.