entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 14 Releases Activity

add a generic proxy application (#72300) #26

Merged
Ghost merged 1 commits from wip/72300-add-proxy-app into main 2023-01-13 11:09:48 +01:00
Conversation 6 Commits 1 Files Changed 6 +346
Ghost commented 2023-01-05 18:49:03 +01:00
First-time contributor
Copy Link
No description provided.
Ghost force-pushed wip/72300-add-proxy-app from c222b4cbd4 to b737a11df9 2023-01-08 13:07:44 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from b737a11df9 to a806807003 2023-01-08 13:19:06 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from a806807003 to 82877dbfc1 2023-01-08 15:25:02 +01:00 Compare
Ghost changed title from WIP: add a generic proxy application (#72300) to add a generic proxy application (#72300) 2023-01-08 15:26:03 +01:00
Ghost force-pushed wip/72300-add-proxy-app from 82877dbfc1 to 354b6fdd92 2023-01-08 15:27:57 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from 354b6fdd92 to d5de91a270 2023-01-08 16:23:24 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from d5de91a270 to 4ef36c2a71 2023-01-08 16:35:00 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from 4ef36c2a71 to 7350060e94 2023-01-08 16:48:29 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from 7350060e94 to 0ea4e1ac9e 2023-01-08 16:57:49 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from 0ea4e1ac9e to 99ea81a90c 2023-01-08 17:00:20 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from 99ea81a90c to b535541297 2023-01-08 17:22:48 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from b535541297 to 27bb14a36e 2023-01-08 17:33:46 +01:00 Compare
Ghost force-pushed wip/72300-add-proxy-app from 27bb14a36e to 97ef062e6c 2023-01-08 17:48:14 +01:00 Compare
ecazenave commented 2023-01-09 12:34:04 +01:00
Owner
Copy Link

Surpris par l'ajout de upstream_base_url dans le modèle, j'imaginais un passe plat vraiment générique qui ne nécessite pas une nouvelle instance de connecteur à chaque fois qu'on veut taper à un nouvel endroit.
Je pense que ça va en freiner l'usage pour des trucs genre https://dev.entrouvert.org/issues/62340.

D'un autre coté répartir les choses sur plusieurs instances de connecteur facilitera les sessions de support technique and co. Mon cœur balance.

Surpris par l'ajout de upstream_base_url dans le modèle, j'imaginais un passe plat vraiment générique qui ne nécessite pas une nouvelle instance de connecteur à chaque fois qu'on veut taper à un nouvel endroit. Je pense que ça va en freiner l'usage pour des trucs genre https://dev.entrouvert.org/issues/62340. D'un autre coté répartir les choses sur plusieurs instances de connecteur facilitera les sessions de support technique and co. Mon cœur balance.
fpeters commented 2023-01-09 12:48:52 +01:00
Owner
Copy Link

Perso pour avoir une instance par service connecté, ce qui permettra d'intégrer à un moment le suivi de disponibilité.

Perso pour avoir une instance par service connecté, ce qui permettra d'intégrer à un moment le suivi de disponibilité.
Ghost commented 2023-01-09 14:08:36 +01:00
Author
First-time contributor
Copy Link

Surpris par l'ajout de upstream_base_url dans le modèle, j'imaginais un passe plat vraiment générique qui ne nécessite pas une nouvelle instance de connecteur à chaque fois qu'on veut taper à un nouvel endroit.
Je pense que ça va en freiner l'usage pour des trucs genre https://dev.entrouvert.org/issues/62340.

Je vois.

Mais mon idée/besoin c'est d'avoir un proxy en place pour des applications tierces compatibles Publik. Le but ici est qu'une fois l'instance proxy en place, le CPF utilise des appels à Passerelle classiques. Il n'a plus à taper d'URL bizarre du genre https://login:pass@api.appli.fr:8088/ws/foo, mais plutôt des {{passerelle_url}}proxy/appli/foo

Sur Passerelle on voit alors dans proxy/appli tout ce qui transite au niveau de cette appli, sans mélange avec autre chose. Et on gagne aussi comme dit Frédéric tout ce qui sera évolution sur BaseResource (suivi de dispo, etc).

S'il y a "n" applis, il faut effectivement instancier "n" proxy, mais ça me semble pas un travail pénible (on a rarement n > 10).

> Surpris par l'ajout de upstream_base_url dans le modèle, j'imaginais un passe plat vraiment générique qui ne nécessite pas une nouvelle instance de connecteur à chaque fois qu'on veut taper à un nouvel endroit. > Je pense que ça va en freiner l'usage pour des trucs genre https://dev.entrouvert.org/issues/62340. Je vois. Mais mon idée/besoin c'est d'avoir un proxy en place pour des applications tierces compatibles Publik. Le but ici est qu'une fois l'instance proxy en place, le CPF utilise des appels à Passerelle classiques. Il n'a plus à taper d'URL bizarre du genre https://login:pass@api.appli.fr:8088/ws/foo, mais plutôt des {{passerelle_url}}proxy/appli/foo Sur Passerelle on voit alors dans proxy/appli tout ce qui transite au niveau de cette appli, sans mélange avec autre chose. Et on gagne aussi comme dit Frédéric tout ce qui sera évolution sur BaseResource (suivi de dispo, etc). S'il y a "n" applis, il faut effectivement instancier "n" proxy, mais ça me semble pas un travail pénible (on a rarement n > 10).
bdauvergne commented 2023-01-10 12:39:42 +01:00
Owner
Copy Link

C'est tout bon pour moi, j'ai transformé les tests pour utiliser responses pour déprécier l'usager des mocks tout moche.

C'est tout bon pour moi, j'ai transformé les tests pour utiliser responses pour déprécier l'usager des mocks tout moche.
bdauvergne force-pushed wip/72300-add-proxy-app from 9486c8c2f4 to f86a8a5e99 2023-01-10 12:41:18 +01:00 Compare
bdauvergne approved these changes 2023-01-10 12:41:45 +01:00
Ghost commented 2023-01-10 14:58:14 +01:00
Author
First-time contributor
Copy Link

C'est tout bon pour moi, j'ai transformé les tests pour utiliser responses pour déprécier l'usager des mocks tout moche.

J'y comprends plus rien à mes tests mais ça roule :)

Va nous falloir trouver un relecteur-validateur tierce maintenant :)

> C'est tout bon pour moi, j'ai transformé les tests pour utiliser responses pour déprécier l'usager des mocks tout moche. J'y comprends plus rien à mes tests mais ça roule :) Va nous falloir trouver un relecteur-validateur tierce maintenant :)
Ghost force-pushed wip/72300-add-proxy-app from f86a8a5e99 to adb08a9ef8 2023-01-13 10:36:36 +01:00 Compare
Ghost commented 2023-01-13 10:37:18 +01:00
Author
First-time contributor
Copy Link

J'ai tout mis dans un seul commit.

J'ai tout mis dans un seul commit.
Ghost merged commit adb08a9ef8 into main 2023-01-13 11:09:48 +01:00
Ghost deleted branch wip/72300-add-proxy-app 2023-01-13 11:09:48 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#26
No description provided.
Delete Branch "wip/72300-add-proxy-app"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?