chrispanag
13 Octubre, 2019 10:55
1
Hola a todos,
tras actualizar nuestro foro a la versión 2.4.0beta6, notamos que varias categorías redirigen a una página de error 404.
Esta llamada devuelve un 404: https://rembetiko.gr/c/%CE%B8%CE%AD%CF%83%CE%B5%CE%B9%CF%82-%CE%B1%CE%BD%CF%84%CE%B9%CE%B8%CE%AD%CF%83%CE%B5%CE%B9%CF%82/find_by_slug.json
Si esto ayuda, en algún momento del pasado, cambiamos nuestros slugs de codificados a ASCII . Pero parece que los slugs de las categorías nunca cambiaron. Intentamos cambiar los slugs de ASCII a codificados, pero nada cambió. (eliminamos la caché, etc.)
Puedes ver el problema en nuestra instancia de Discourse: https://rembetiko.gr
¿Alguna sugerencia? Sospechamos que se trata de algún tipo de error…
5 Me gusta
Interesante, ¿alguna idea @falco ?
3 Me gusta
Falco
13 Octubre, 2019 17:58
3
Oh, es posible que haya olvidado los identificadores de categoría en mi último parche, donde solo corregí los identificadores de temas.
Lo revisaré el lunes.
6 Me gusta
azawawi
15 Octubre, 2019 08:15
5
Falco:
en mi último parche
¿Es este el commit al que te refieres?
committed 03:38PM - 11 Oct 19 UTC
When an admin changes the site setting slug_generation_method to
encoded, we we… ren't really encoding the slug, but just allowing non-ascii
characters in the slug (unicode).
That brings problems when a user posts a link to topic without the slug, as
our topic controller tries to redirect the user to the correct URL that contains
the slug with unicode characters. Having unicode in the Location header in a
response is a RFC violation and some browsers end up in a redirection loop.
Bug report: https://meta.discourse.org/t/-/125371?u=falco
This commit also checks if a site uses encoded slugs and clear all saved slugs
in the db so they can be regenerated using an onceoff job.
2 Me gusta
Falco
15 Octubre, 2019 18:51
6
Sí, esa.
Estoy arreglando lo que está roto en las categorías en este PR:
master ← more-encoded-slug-fixes
merged 08:08PM - 16 Oct 19 UTC
A bunch of extra places to consider when dealing with encoded slugs.
Para ser honesto, en cierto modo me arrepiento de la corrección inicial, ya que toca demasiadas partes del código para manejar las URLs codificadas.
4 Me gusta
¿Entonces esto ya está totalmente arreglado o sigue en desarrollo?
2 Me gusta
Tengo exactamente el mismo problema en mi foro 2.4.0.beta6 [a37dafdd4d]
El error 404 parece aparecer solo en las categorías que tienen “acentos” en su texto.
Espero que puedan lanzar el parche y actualizar pronto, ya que esto interrumpe el correcto funcionamiento del foro.
¡Gracias!
1 me gusta
Falco
16 Octubre, 2019 20:55
9
@TroyaNO @azawawi @chrispanag @Jeremie_Leroy Acabo de fusionar otra ronda de correcciones, por favor actualiza y échale un vistazo.
Además, cierro esto a favor de Topic with Japanese in URL doesn't redirect if URL doesn't perfectly match
4 Me gusta
, después de actualizar hace unos minutos a: 2.4.0.beta6 [a37dafdd4d]