Projet

Général

Profil

Bug #6539

Option to avoid checking https certificate correctness

Ajouté par Frédéric Péters il y a environ 9 ans. Mis à jour il y a environ 9 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
Début:
17 février 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

In development environment we may not have valid ssl certificates, it would be nice in that situation if ssl checks could be turned off in authentic.


Fichiers

Révisions associées

Révision f8543c96 (diff)
Ajouté par Frédéric Péters il y a environ 9 ans

misc: raise an urllib2.HTTPError if get_url returns a non 200 response (#6539)

Révision 783977b6 (diff)
Ajouté par Frédéric Péters il y a environ 9 ans

misc: add a settings option to disable https ssl checks (#6539)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 9 ans

The first patch enhances get_pycurl_url() to raise an error on http error codes (much like the non-http code path of urllib2 does); the second patch adds an option to not verify certificates.

#2

Mis à jour par Thomas Noël il y a environ 9 ans

Ack pour moi, mais validation finale par Benj si possible.

#4

Mis à jour par Frédéric Péters il y a environ 9 ans

  • Fichier 0002-misc-add-a-settings-option-to-disable-https-ssl-chec.patch supprimé
#5

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

Je ne sais pas à que point c'est pressé mais je verrai bien d'enlever le module authentic2.http_utils pour n'utiliser que requests à la place, et simplement faire des requests.get(url, verify=app_settings.A2_VERIFY_SSL) là où c'est nécessaire. Dans l'interim ces deux patchs sont nickels.

#6

Mis à jour par Frédéric Péters il y a environ 9 ans

  • Statut changé de Nouveau à Résolu (à déployer)

Thanks, I'll thus push those now, and leave the switch to python-requests for another one (I'll open a ticket about that).

#7

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Version cible mis à 2.1.12
#8

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF