Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

problème avec les urls des tags contenant un point #212

Open
brunob opened this issue Jul 11, 2019 · 5 comments

Comments

Projects
None yet
2 participants
@brunob
Copy link
Member

commented Jul 11, 2019

Signalé par @Fil ici https://seenthis.net/messages/792213

Corrigé en prod en rétablissant la rewrite suivante dans conf/nginx/rewrites.conf :

  location ~ ^/tag/ {
    rewrite ^/(tag/.*)$      /spip.php                             last;
  }

Sans ça, on ne passe pas du tout dans la partie décodage de la fonction urls_seenthis_dist(), j'ai tenté de désactivé toutes les règles du htaccess en local (car je travaille avec apache) et cela n'a aucun effet. Je sèche pour l'instant...

@Fil

This comment has been minimized.

Copy link
Member

commented Jul 11, 2019

@brunob

This comment has been minimized.

Copy link
Member Author

commented Jul 11, 2019

Côté apache, il semble que ça vienne de la rewrite rule des urls propres du htaccess, qui "refuse" les points cf https://core.spip.net/projects/spip/repository/entry/branches/spip-3.2/htaccess.txt

Avec ça dans mon htaccess RewriteRule ^(tag/.*)$ spip.php [QSA,L] ça fonctionne bien.

Donc, si on veut faire simple, on rétablit cette rewrite rule dans dans le htaccess fourni dans la repo, et on maj la page du wiki pour la recoller dans les rewrites nginx.

À voir s'il faut en faire de même pour les rewrites suivantes :

RewriteRule ^(tags/.*)\$$     $1?page=mot_fin [QSA,L]
RewriteRule ^(tags/.*)\*$ $1?page=mot_flou [QSA,L]

Perso j'accède bien à /tags, mais je ne sais plus à quoi correspond /tags/??? => https://seenthis.net/tags/expulsion$ ?

@brunob brunob self-assigned this Jul 11, 2019

brunob added a commit to seenthis/seenthis that referenced this issue Jul 12, 2019

retour des rewrites pour les pages des tags
car les tags peuvent contenir des points et SPIP ne prend pas ça en charge dans la rewrite des urls propres

ref seenthis/seenthis_squelettes#212
@brunob

This comment has been minimized.

Copy link
Member Author

commented Jul 12, 2019

Je viens de rétablir les 3 rewrites pour mot, mot_fin et mot_flou dans le htaccess, mais je doute car en passant par l'url tag/ (sans s) ça fonctionne bien, exemple :

https://seenthis.net/tag/d3.js* <= mot flou ok
https://seenthis.net/tags/d3.js* <= ko 404

@martinarnaud peut nous en dire plus à ce sujet ?

@brunob

This comment has been minimized.

Copy link
Member Author

commented Jul 12, 2019

Et donc, si ces redirections sont bien nécessaires, il faudra penser à les rétablir côté nginx en prod et sur le wiki https://github.com/seenthis/hebergement/wiki/Configuration-nginx

  location ~ ^/tags/?$ {
    rewrite ^/tags/?$      /spip.php?page=tags                       last;
    rewrite ^/(tags/.*)\$$     /$1?page=mot_fin                      last;
    rewrite ^/(tags/.*)\*$     /$1?page=mot_flou                     last;
  }

@brunob brunob added the bug label Jul 12, 2019

@brunob

This comment has been minimized.

Copy link
Member Author

commented Jul 12, 2019

Pour info, j'ai temporairement rétabli les rewrites nginx de /tags/ et ça ne change rien, https://seenthis.net/tags/d3.js* renvoie toujours une 404...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.