CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

#202 Remove legacy url support #203

Closed nhumblot closed 3 years ago

nhumblot commented 3 years ago

Fix #202

fcamblor commented 3 years ago

@nhumblot as-tu une idée de la manière qui permettrait de tester ça avant la prod ?

nhumblot commented 3 years ago

@nhumblot as-tu une idée de la manière qui permettrait de tester ça avant la prod ?

Normalement, si on teste maintenant en prod, on devrait avoir une http 302 sur les urls legacy, je peux fournir plus d'éléments ce soir quand j'aurai un pc.

Édit: la conf a aussi été appliquée sur dev.vitemado.se. On peut aussi tester sur ce domaine.

fcamblor commented 3 years ago

Actuellement, https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt3-gironde/age-plus75/ retourne une 304 Not Modified, du coup j'ai un petit doute :-)

nhumblot commented 3 years ago

@fcamblor Ça semble déployé sur le dev (j'ai une 302) mais pas encore la prod (j'ai une 200). :thinking:

[nhumblot@PC-008-SITEFLOW-FR vitemadose-front]$ curl -v https://dev.vitemado.se/dev/centres-vaccination-covid-dpt05-hautes-alpes/age-plus75ans/
*   Trying 146.59.231.206:443...
* Connected to dev.vitemado.se (146.59.231.206) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*  CAfile: /etc/pki/tls/certs/ca-bundle.crt
*  CApath: none
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_128_GCM_SHA256
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=dev.vitemado.se
*  start date: Apr 12 10:50:50 2021 GMT
*  expire date: Jul 11 10:50:50 2021 GMT
*  subjectAltName: host "dev.vitemado.se" matched cert's "dev.vitemado.se"
*  issuer: C=US; O=Let's Encrypt; CN=R3
*  SSL certificate verify ok.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x560de75dfbf0)
> GET /dev/centres-vaccination-covid-dpt05-hautes-alpes/age-plus75ans/ HTTP/2
> Host: dev.vitemado.se
> user-agent: curl/7.76.1
> accept: */*
> 
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* Connection state changed (MAX_CONCURRENT_STREAMS == 250)!
< HTTP/2 302 
< access-control-allow-origin: https://vitemadose.covidtracker.fr
< alt-svc: h3-32=":443"; ma=2592000,h3-29=":443"; ma=2592000
< cache-control: max-age=0, no-cache, no-store, must-revalidate
< location: /dev/centres-vaccination-covid-dpt05-hautes-alpes/
< referrer-policy: no-referrer-when-downgrade
< server: Caddy
< strict-transport-security: max-age=31536000;
< x-content-type-options: nosniff
< x-frame-options: deny
< content-length: 0
< date: Fri, 14 May 2021 16:06:25 GMT
< 
* Connection #0 to host dev.vitemado.se left intact
nhumblot commented 3 years ago

@fcamblor Dans State.ts : export type CodeTrancheAge = 'plus75ans';.

L'url legacy c'était bien https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt3-gironde/age-plus75ans/ (j'ai une 302) et non https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt3-gironde/age-plus75/ (j'ai une 200) que tu as testée ?

curl -v https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt3-gironde/age-plus75/ => 200

curl -v https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt3-gironde/age-plus75ans/ => 302

< HTTP/2 302 
< date: Fri, 14 May 2021 16:18:59 GMT
< content-length: 0
< location: /centres-vaccination-covid-dpt3-gironde/

Tu peux tester dans ton navigateur : https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt33-gironde/age-plus75ans/ te renvoie sur https://vitemadose.covidtracker.fr/centres-vaccination-covid-dpt33-gironde/

fcamblor commented 3 years ago

Oh oui my bad, j'avais omis le ~ans :-)