#37 Erreur dans les conversions de routes du forum

Closed
opened 7 months ago by Darks · 7 comments
Darks commented 7 months ago

Les routes du forum sont mal interprétées.

Exemple, je créé le topic /forum/news/calc/2. Je peux aussi y accéder par /forum/news/projects/2. Comportement attendu → 404.

De plus, url_for('forum_topic', topic.forum, topic.id) retourne quelque chose de la forme /forum/news/calc/<Topic: #2>, ce qui casse les liens. Comportement attendu : /forum/news/calc/2-slug-du-topic.

Les routes du forum sont mal interprétées. Exemple, je créé le topic `/forum/news/calc/2`. Je peux aussi y accéder par `/forum/news/projects/2`. Comportement attendu → 404. De plus, `url_for('forum_topic', topic.forum, topic.id)` retourne quelque chose de la forme `/forum/news/calc/<Topic: #2>`, ce qui casse les liens. Comportement attendu : `/forum/news/calc/2-slug-du-topic`.
Darks added the
bug
label 7 months ago
Darks self-assigned this 7 months ago
Lephenixnoir was assigned by Darks 7 months ago
Darks removed their assignment 7 months ago
Darks commented 7 months ago
Owner

J'ai corrigé le point n°2 dans le commit b5e875e136

Pour le reste, @Lephenixnoir tu pourra essayer de regarder ?

J'ai corrigé le point n°2 dans le commit b5e875e136 Pour le reste, @Lephenixnoir tu pourra essayer de regarder ?
Lephenixnoir commented 7 months ago
Owner

J'ai pas l'environnement de dev ici, mais je devrais pouvoir le faire.

J'ai pas l'environnement de dev ici, mais je devrais pouvoir le faire.
Eragon commented 5 months ago
Collaborator

Up

Up
Lephenixnoir commented 4 months ago
Owner

J'ai attaqué sérieusement ce bug aujourd'hui. Pas encore gagné mais c'est un début.

Déjà le problème d'origine selon lequel les topics seraient accessibles sous d'autres noms de forum n'a plus l'air de se poser.

Je confirmerai plus en détail quand j'arriverai à créer des nouveaux topics, ce qui nécessite pour l'instant de se connecter donc de configurer LDAP... ce qui est un peu la merde. @Darks détails désirés.

J'ai attaqué sérieusement ce bug aujourd'hui. Pas encore gagné mais c'est un début. Déjà le problème d'origine selon lequel les topics seraient accessibles sous d'autres noms de forum n'a plus l'air de se poser. Je confirmerai plus en détail quand j'arriverai à créer des nouveaux topics, ce qui nécessite pour l'instant de se connecter donc de configurer LDAP... ce qui est un peu la merde. @Darks détails désirés.
Darks commented 4 months ago
Owner

Méthode si tu veux pas te faire chier : ajouter USE_LDAP = False à la classe LocalConfig

Méthode si tu veux (un peu) te faire chier : https://gitea.planet-casio.com/devs/VPS-config/src/branch/master/ldap

Je précise que si t'utilise pas systemd faut adapter le script que je t'ai linké. :wink:

Méthode si tu veux pas te faire chier : ajouter `USE_LDAP = False` à la classe `LocalConfig` Méthode si tu veux (un peu) te faire chier : https://gitea.planet-casio.com/devs/VPS-config/src/branch/master/ldap Je précise que si t'utilise pas systemd faut adapter le script que je t'ai linké. :wink:
Lephenixnoir commented 4 months ago
Owner

Merci ! J'ai configuré le LDAP, ça ne coûte rien.

Je suppose que j'ai mis trop longtemps à étudier la question... en tous cas je ne peux pas reproduire le bug. @Darks s'il te reste un moment pour vérifier que j'ai bien cherché, sinon on peut fermer l'isue.

Merci ! J'ai configuré le LDAP, ça ne coûte rien. Je suppose que j'ai mis trop longtemps à étudier la question... en tous cas je ne peux pas reproduire le bug. @Darks s'il te reste un moment pour vérifier que j'ai bien cherché, sinon on peut fermer l'isue.
Darks commented 4 months ago
Owner

Je dois t'avouer que je ne sais plus d'où venait exactement le bug. Y'a du y'avoir un fix entre temps. M'enfin si ça marche, je ferme.

Merci pour ton aide 🙂

Je dois t'avouer que je ne sais plus d'où venait exactement le bug. Y'a du y'avoir un fix entre temps. M'enfin si ça marche, je ferme. Merci pour ton aide 🙂
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.