Project

General

Profile

Development #70632

Dépréciation: DeprecationWarning: please use dns.resolver.resolve() instead

Added by Benjamin Dauvergne 3 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
25 October 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

  /var/lib/jenkins/workspace/authentic/src/authentic2/utils/evaluate.py:98: DeprecationWarning: please use dns.resolver.resolve() instead
    answers = dns.resolver.query(domain, 'A', lifetime=1)
</pew>

Files

Associated revisions

Revision 427021b5 (diff)
Added by Benjamin Dauvergne 3 months ago

misc: do not use dns.resolver.query on later versions of dnspython (#70632)

History

#1

Updated by Benjamin Dauvergne 3 months ago

#2

Updated by Thomas Noël 3 months ago

Pas bien compris pourquoi tu ajoutes dans tox.ini le filtre sur les warnings :

error:.*please use dns.resolver.resolve:DeprecationWarning:

alors que justement le patch évite ce warning, non ?

#3

Updated by Benjamin Dauvergne 3 months ago

Thomas Noël a écrit :

Pas bien compris pourquoi tu ajoutes dans tox.ini le filtre sur les warnings :

error:.*please use dns.resolver.resolve:DeprecationWarning:

alors que justement le patch évite ce warning, non ?

C'est pour être sûr qu'il n'est plus là. C'est plus simple que de scruter les logs.

#4

Updated by Thomas Noël 3 months ago

  • Status changed from Solution proposée to Solution validée

La syntaxe du tox.ini m'échappait, je viens de comprendre que c'est le filtre est un antifiltre.

#5

Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 427021b5b2462b5dcf28c3b1e82646c632b8278b
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Tue Oct 25 07:44:12 2022 +0200

    misc: do not use dns.resolver.query on later versions of dnspython (#70632)
#6

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF