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

[POST] proteger les underscores dans les urls #581

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Apr 2, 2024

Description

🎸 Machina s'appuie sur la librairie markdown2 pour le rendu des contenus des messages (post.content)
🎸 L'avantage principal est de nettoyer le texte en remplaçant le code html saisi par l'utilisateur, par [HTML_REMOVED]
🎸 Cette substitution est utiliser dans la protection contre les spams.

🦺 Cependant, markdown2 converti les _ en balise markdown (<em>), ayant pour effet de casser le traitement de l'urlizer sur une url contenant des underscore.
🦺 L'ajout du package code-friendly dans la configuration de markdown2 résout ce problème.

Type de changement

🪲 Correction de bug (changement non cassant qui corrige un problème).
🚧 technique

point d'attention

  • Le setup par défaut des settings de machina pour MACHINA_MARKUP_LANGUAGE
    est ('machina.core.markdown.markdown', {'safe_mode': True, 'extras': {'break-on-newline': True}})

  • La surcharge de ce setup dans lacommunaute necessite de reprendre toute la conf initiale plus les extra-packages désirés

  • Le package nofollow est ajouté à l'occasion, bien que passé en paramètre dans l'appel à urlize dans la méthode urlizetrunc_target_blank

@vincentporte vincentporte added the hot fix Pull requests that fix a bug to fix as soon as possible label Apr 2, 2024
@vincentporte vincentporte removed the request for review from leo-naeka April 3, 2024 14:16
@vincentporte vincentporte self-assigned this Apr 3, 2024
@vincentporte vincentporte merged commit 7f7410c into master Apr 3, 2024
4 checks passed
@vincentporte vincentporte deleted the vincentporte/markdown2_code_friendly branch April 3, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hot fix Pull requests that fix a bug to fix as soon as possible
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant