search

italia / docs.italia.it

The source code that powers docs.italia.it
MIT License
22 stars 12 forks source link

Le pagine inesistenti con un path che corrisponde ad un documento non tornano 404 #218

Closed pdavide closed 4 years ago

pdavide commented 6 years ago

Tutte le pagine inesistenti con path: /<publisher>/<progetto>/<documento>/<lingua>/<versione>/<pagina inesistente> non restituiscono un errore ma la homepage del documento.

pdavide commented 5 years ago

Sembra che il bug sia ancora (o di nuovo) presente.

Amanit commented 4 years ago

@pdavide non riesco a riprodurre il bug

pdavide commented 4 years ago

@Amanit visita https://docs-italia-staging.teamdigitale.it/AgID/documenti-in-consultazione/lg-cert-regionali/it/bozza/nonexistent.html

yakky commented 4 years ago

@Amanit secondo me è questa riga qua https://github.com/italia/docs.italia.it/blob/italia-dev/docker/nginx/sites-available/readthedocs-main.conf#L99

se vedi usa esplicitamente degli index.html se $path non esiste

puoi verificare se senza togliendo il riferimento a index.html torna correttamente un 404

ed eventualmente possiamo capire se gestire in modo diverso il fallback su index.html per la root

Amanit commented 4 years ago

@pdavide abbiamo modificato i settings sul server di staging. Potrebbe testare?

yakky commented 4 years ago

@pdavide @francescozaia il fix per questa è deployato su staging