Refactoring de la gestion des options headers/datas de recuperer_url() #4986

Merged
cerdic merged 2 commits from fix_redirections_en_get into master 2 days ago
cerdic commented 2 weeks ago
Owner

Lors d'une redirection apres un POST, on doit faire un GET sauf si la redirection est de type 307 ou 308 ce qui indique explicitement de garder la méthode d'origine
https://developer.mozilla.org/en-US/docs/Web/HTTP/Status

Le refactoring permet de mieux gerer les sequences

  • POST url => 301 + Location url2 => GET sur url2
  • POST url => 307 sur url2 => POST sur url2 => 302 sur url3 => GET sur url3

Le tout sans perdre les headers d'origine (type User agent, accept, cookies...) et en transmettant les datas au hit suivant uniquement si on a une 307 ou 308

En consequence on refabrique les donnes formattes a chaque redirection si besoin a partir des options headers+datas dont la derniere est eventuellement videe en cours de route

Lors d'une redirection apres un POST, on doit faire un GET sauf si la redirection est de type 307 ou 308 ce qui indique explicitement de garder la méthode d'origine https://developer.mozilla.org/en-US/docs/Web/HTTP/Status Le refactoring permet de mieux gerer les sequences * POST url => 301 + Location url2 => GET sur url2 * POST url => 307 sur url2 => POST sur url2 => 302 sur url3 => GET sur url3 Le tout sans perdre les headers d'origine (type User agent, accept, cookies...) et en transmettant les datas au hit suivant uniquement si on a une 307 ou 308 En consequence on refabrique les donnes formattes a chaque redirection si besoin a partir des options headers+datas dont la derniere est eventuellement videe en cours de route
cerdic added 1 commit 2 weeks ago
0e87daa7b9 Refactoring de la gestion des options headers/datas de recuperer_url() pour mieux gerer les sequences POST => 301 + Location => GET sur la nouvelle URL
cerdic added 1 commit 2 weeks ago
marcimat approved these changes 1 week ago
cerdic added 2 commits 2 days ago
5ba670f027 Refactoring de la gestion des options headers/datas de recuperer_url() pour mieux gerer les sequences POST => 301 + Location => GET sur la nouvelle URL
cerdic merged commit ca21acd446 into master 2 days ago
cerdic deleted branch fix_redirections_en_get 2 days ago

Reviewers

marcimat approved these changes 1 week ago
The pull request has been merged as ca21acd446.
Sign in to join this conversation.
Loading…
There is no content yet.