Development #24713
Supporter les appels vers l'idp FC via un proxy
100%
Description
La connexion à FC derrière un proxy n'est pas possible.
Fichiers
Demandes liées
Révisions associées
add fc requests proxies support (#24713)
Historique
Mis à jour par Josué Kouka il y a presque 6 ans
- Fichier 0001-add-requests-proxies-support-24713.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Josué Kouka il y a presque 6 ans
- Fichier
0001-add-requests-proxies-support-24713.patchsupprimé
Mis à jour par Josué Kouka il y a presque 6 ans
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Solution proposée à En cours
Coté passerelle le setting s'appelle REQUEST(S)_PROXIES, on doit faire pareil (avec le S à REQUEST donc).
Tente de faire des tests si tu voir comment (dans passerelle c'est plus simple il y a une abstraction autour de requests).
PS: en fait on a ça aussi dans authentic2-auth-fc, voir authentic2_auth_fc.utils.requests_retry_session, c'est là que tu dois appliquer ta modification et tu pourras juste faire un test en créant un session via cette fonction d'aide en testant un get() quelconque.
Mis à jour par Josué Kouka il y a presque 6 ans
- Fichier 0001-add-fc-requests-proxies-support-24713.patch 0001-add-fc-requests-proxies-support-24713.patch ajouté
- Statut changé de En cours à Solution proposée
Un patch qui suit ton commentaire. Je n'ai pas écris de test avec un appel get parce que je n'ai pas trouvé de quoi faire un assert pour valider que la vraie url appelé est celle du proxy. Je me suis assuré que le proxy est bien pris en compte.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Josué Kouka a écrit :
Un patch qui suit ton commentaire.
Je n'ai pas écris de test avec un appelgetparce que je n'ai pas trouvé de quoi faire unassertpour valider que la vraie url appelé est celle du proxy.
Tu as la réponse dans les tests passerelle de la même fonctionnalité (je l'indiquais au dessus) faire un mock de requests.Session.request.
Mis à jour par Josué Kouka il y a presque 6 ans
- Fichier 0001-add-fc-requests-proxies-support-24713.patch 0001-add-fc-requests-proxies-support-24713.patch ajouté
Benjamin Dauvergne a écrit :
Josué Kouka a écrit :
Un patch qui suit ton commentaire.
Je n'ai pas écris de test avec un appelgetparce que je n'ai pas trouvé de quoi faire unassertpour valider que la vraie url appelé est celle du proxy.Tu as la réponse dans les tests passerelle de la même fonctionnalité (je l'indiquais au dessus) faire un mock de requests.Session.request.
Ok. un patch avec un test similaire.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Josué Kouka il y a presque 6 ans
- Statut changé de Solution validée à Résolu (à déployer)
- % réalisé changé de 0 à 100
commit deebd83a60fa6d061bb98590b72d369f1d62ae0c (HEAD -> master, origin/master, origin/HEAD)
Author: Josue Kouka <jkouka@entrouvert.com>
Date: Tue Jun 26 11:47:18 2018 +0200
add fc requests proxies support (#24713)
Mis à jour par Thomas Noël il y a presque 6 ans
Mmmmh... moi j'ai pas bien compris le ack : pourquoi A2_FC_REQUESTS_PROXIES alors qu'on voulait juste utiliser REQUESTS_PROXIES ?
Mis à jour par Josué Kouka il y a presque 6 ans
Thomas Noël a écrit :
Mmmmh... moi j'ai pas bien compris le ack : pourquoi A2_FC_REQUESTS_PROXIES alors qu'on voulait juste utiliser REQUESTS_PROXIES ?
Parce que c'est un parametre dans le scope du plugin FC. Ce qui implique le suffixe A2_FC. (Je ne sais pas trop si ça répond à ta question).
Mis à jour par Frédéric Péters il y a presque 6 ans
Parce que c'est un parametre dans le scope du plugin FC. Ce qui implique le suffixe A2_FC. (Je ne sais pas trop si ça répond à ta question).
Mais ça n'a pas de sens dans Authentic d'avoir à définir un paramétrage de proxy par application django.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
Je suis ok pour qu'on le nomme REQUESTS_PROXIES en le prenant directement dans settings sans passer par app_settings (on va dire que c'est global).
Mis à jour par Thomas Noël il y a presque 6 ans
J'ai fait #24929 pour laisser le présent ticket fermé.
Mis à jour par Thomas Noël il y a presque 6 ans
- Lié à Bug #24929: utiliser settings.REQUESTS_PROXIES ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
add fc requests proxies support (#24713)