Nouvelle dépêche avec un lien type http://localhost/ééé
-> HTTP 500

FATAL -- : [xxx] URI::InvalidURIError (URI must be ascii only "http://localhost/\u{e9}\u{e9}\u{e9}"):

Il y a pire, si on réorganise la dépêche et que l'on insère un lien invalide -> HTTP 500 + perte du format markdown (l'entrée news_versions est conservée, les entrées paragraphs sont perdues).

Pour faciliter la modération (notamment la détection du spam), lors d'une visioconf de l'équipe de modération, il a été suggéré de pouvoir affadir un fil de discussion (dans le but de le rendre moins attractif). La version dure serait d'empêcher tout commentaire supplémentaire en dessous, la version douce de jouer de la CSS pour affadir l'affichage et suggérer plus délicatement de passer à autre chose.

Pour faciliter la modération (notamment la détection du spam), lors d'une visioconf de l'équipe de modération, il a été demandé d'avoir une page listant les derniers commentaires sur le site (probablement titre, lien, texte du commentaire, et pagination sur l'index).

Une erreur qui revient régulièrement, avec quelques occurrences par jour :

I, [2021-05-12T09:35:47.595799 #12069]  INFO -- : [uuid] Started GET "/board/index.tsv" for xx.xx.xx.xx at 2021-05-12 09:35:47 +0200
I, [2021-05-12T09:35:47.597218 #12069]  INFO -- : [uuid] Processing by BoardsController#show as TSV
I, [2021-05-12T09:35:47.639061 #12069]  INFO -- : [uuid]   Rendering boards/show.tsv.erb
I, [2021-05-12T09:35:47.658799 #12069]  INFO -- : [uuid]   Rendered boards/show.tsv.erb (19.6ms)
I, [2021-05-12T09:35:47.659080 #12069]  INFO -- : [uuid] Completed 500 Internal Server Error in 62ms (ActiveRecord: 0.0ms)
F, [2021-05-12T09:35:47.660641 #12069] FATAL -- :

Quelle que soit la tribune (libre, de rédaction, de modération, d'une dépêche, etc.), il serait intéressant de pouvoir virer une entrée dessus en cas de besoin (le spam étant le cas le plus fréquent) : pouvoir choisir la tribune cible et pouvoir remplacer un message donné sur cette tribune par Ce commentaire a été supprimé par l’équipe de modération. (idéalement avec une modification de façon que seule l'équipe du site soit capable d'écrire un tel message). Au besoin on peut (…)

Les étiquettes masquées n'ont pas vocation à être indexées par les moteurs de recherche (que la raison soit un doublon, une meilleure étiquette existante, une abréviation, du franglais, etc., etc.)

le epub https://linuxfr.org/users/tonio/journaux/s-occuper-pendant-les-vacances-yunohost-et-autohebergement.epub ne passe pas le validateur epubcheck (j'ai fait le test en distant via http://validator.idpf.org/ puis en local) :

$ epubcheck s-occuper-pendant-les-vacances-yunohost-et-autohebergement.epub
Vérifications faites en utilisant les règles de la version epub 3.2.
ERROR(MED-004): s-occuper-pendant-les-vacances-yunohost-et-autohebergement.epub/EPUB/(-1,-1): L'en-tête du fichier image pourrait être corrompu.
ERROR(PKG-021): s-occuper-pendant-les-vacances-yunohost-et-autohebergement.epub/EPUB/(-1,-1): Fichier image corrompu rencontré.

Vérification terminée avec des erreurs
Messages: 0 fatale / 2 erreurs / 0 avertissement / 0 info

EPUBCheck terminé

Faisons le test avec un epub de référence (Children's Literature (…)

Une campagne récente de ré-étiquetage massif a montré un effet des étiquettes populaires dans le bas de page :

https://pix.toile-libre.org/upload/original/1610822726.png

(l'évolution de la courbe correspond en fait à l'évolution temps de rendu de _layouts/_footer.html.haml )

Bref on va avoir besoin d'une mise en cache (pour éviter la sensibilité sur les étiquetages du dernier mois).

Dans app/models/comment.rb

  def wikify_body
    nofollow = user.account.karma < 50
    self.body = wikify(wiki_body, nofollow: nofollow)
  end

du coup toute édition (y compris par un admin) d'un commentaire appartenant à un compte supprimé (entrée dans accounts supprimée mais entrée encore présente dans users) échoue :

Completed 500 Internal Server Error in 16ms (ActiveRecord: 1.7ms)
NoMethodError (undefined method `karma' for nil:NilClass):
app/models/comment.rb:42:in `wikify_body'
app/controllers/comments_controller.rb:64:in `update'

A priori s'il n'y a plus (…)

Éviter que le Markdown ne produise de l'HTML invalide

• Hyperlien décrit, mais sans URL

[Description d'un lien mais pas de lien]()

avec href vide invalide

<a href="">Description d'un lien mais pas de lien</a>

• Tableau mais sans contenu

|Colonne 1|Colonne 2|
|---------|---------|

avec un <tbody> vide invalide

<table>
<thead>
<tr>
<th>Colonne 1</th>
<th>Colonne 2</th>
</

• documenter le schéma redis (fait, reste à commiter)
• ajouter un test pour vérifier la conformité au schéma (en cours)
• nettoyer ce qui dépasse au lance-flamme sur alpha/prod (en cours)

(signalement via https://linuxfr.org/forums/general-general/posts/les-tags-populaires-de-linuxfr-dans-le-pied-de-page )

Dans les tags populaires en bas de page, on affiche les tags non masqués certes (donc pas 'grosconnard'), mais aussi ceux des contenus masqués (exemple 'banque' qui est un tag public acceptable, mais qui traine sur les spams et est donc sur-évalué).

Version brute

• http://example.invalidhttp://example.invalid (double URL) -> OK, au moins ça se voit que ça marche pas
• http://https://example.invalid ou http://ftp://example.invalid (double schéma) -> OK, au moins ça se voit
• http%20://example.invalid (espace excédentaire) -> OK, non cliquable
• http:/example.invalid (barre oblique manquante) -> OK, non cliquable
• news://example.invalid , mailto://example@example.invalid ou xmpp://example@example.invalid (deux barres obliques excédentaires) => OK, au moins ça se voit que ça marche pas
• htttp://example.invalid ou htpp://example.invalid ou htt://example.invalid ou http:example.invalid (mauvais schéma) -> OK, non cliquables
• absence de (…)

Dans l'espace de rédaction :

• après le titre « Dépêches en cours de modération », ajouter une phrase du type « L'accès à l'espace de modération est réservé aux modérateurs et aux administrateurs. Seuls les titres des dépêches en modération sont visibles des autres utilisateurs authentifiés. »
• rendre les titres cliquables pour les modérateurs/admins (et pas pour les autres donc)
• ajouter dans le titre le nombre de dépêches concernées ? (genre « Dépêches en cours de modération (2) »), et ajouter un titre « Dépêches en cours de (…)

Tentative d'ajout d'un lien dans une dépêche, avec comme URL https://riot.im/app/#/room/#

INFO -- : Started POST "/redaction/links" for xx.xx.xx.xx at 2017-07-20 23:44:14 +0200
INFO -- : Processing by Redaction::LinksController#create as HTML
INFO -- :   Parameters: {"utf8"=>"✓", "authenticity_token"=>"[FILTERED]", "news_id"=>"36926", "link"=>{"title"=>"Coin", "url"=>"https://riot.im/app/#/room/#", "lang"=>"fr"}}
INFO -- : Completed 500 Internal Server Error in 10ms (ActiveRecord: 1.5ms)
FATAL -- : 
URI::InvalidURIError (bad URI(is not URI?): https://riot.im/app/#/room/#):
  app/models/link.rb:33:in `url='
  app/controllers/redaction/links_controller.rb:12:in `create'