Suivi — Tribune Souci indéfini sur le tsv de la tribune

#1991 Posté par Benoît Sibaud (site web personnel) le 12 mai 2021 à 11:24. État de l’entrée : corrigée. Assigné à Benoît Sibaud. Licence CC By‑SA.

Étiquettes : aucune

mai

2021

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 -- : [uuid]   
F, [2021-05-12T09:35:47.660717 #12069] FATAL -- : [uuid] ActionView::Template::Error (undefined method `each_char' for nil:NilClass):
F, [2021-05-12T09:35:47.660881 #12069] FATAL -- : [uuid]     1: <% @boards.reverse_each do |board| %>
F, [2021-05-12T09:35:47.660943 #12069] FATAL -- : [uuid]   
F, [2021-05-12T09:35:47.661011 #12069] FATAL -- : [uuid] app/models/board.rb:104:in `remove_control_chars'

# Creusons un peu

Posté par Benoît Sibaud (site web personnel) le 12 mai 2021 à 14:55. Évalué à 3 (+0/-0).
Sur une IP n°1, en général :
```
 /board/index.tsv HTTP/2.0" 200 <taille> "-" "Go-http-client/2.0"
```
et une fois :
```
 /board/index.tsv HTTP/2.0" 500 2027 "-" "Go-http-client/2.0"
```
(plus des accès vers /board/index.xml mais avec un autre user-agent)

Sur une IP n°2, dans la même seconde :
```
"POST /api/v1/board HTTP/1.1" 200 14 "-" "xxx"
"GET /board/index.tsv HTTP/2.0" 500 2027 "-" "Go-http-client/2.0"
"GET /board/index.tsv HTTP/1.1" 200 19051 "-" "yyy"
```
On aurait pu croire que c'était lié à HTTP/2.0 ou/et client particulier "Go-http-client/2.0"… Mais les logs de mai montrent d'autres cas en HTTP/1.1 et avec un autre user-agent.
Répondre
- [^] # Re: Creusons un peu
  
  Posté par Benoît Sibaud (site web personnel) le 12 mai 2021 à 16:13. Évalué à 3 (+0/-0).
  Sur les 24 occurrences de mai 2021 en stock :
  - 22 montrent un "POST /api/v1/board en 200 ou un POST /board en 201" et un "GET /board/index.tsv en 500" dans la même seconde
  - 1 avec POST suivi d'un GET 500 espacés de ~2s
  - 1 avec POST suivi d'un GET 500 espacés de ~5s
  ça semble une meilleure cause… tendance race condition on essaierait de lire une entrée de la tribune qui ne semble pas encore complètement totalement entièrement disponible ?
  Répondre
  - [^] # Re: Creusons un peu
    
    Posté par Benoît Sibaud (site web personnel) le 12 mai 2021 à 16:40. Évalué à 3 (+0/-0).
    à défaut de reproduire, le correctif suivant doit corriger le souci
```
diff --git a/app/models/board.rb b/app/models/board.rb
index ce365949..6dc2b072 100644
--- a/app/models/board.rb
+++ b/app/models/board.rb
@@ -101,8 +101,10 @@ class Board

   def remove_control_chars(msg)
     sanitized = ""
-    msg.each_char do |char|
-      sanitized << char unless char.ascii_only? and (char.ord < 32 or char.ord == 127)
+    unless msg.nil?
+      msg.each_char do |char|
+        sanitized << char unless char.ascii_only? and (char.ord < 32 or char.ord == 127)
+      end
     end
     return sanitized
   end
```
    Répondre
    - [^] # Re: Creusons un peu
      
      Posté par Benoît Sibaud (site web personnel) le 19 novembre 2022 à 18:43. Évalué à 3 (+0/-0).
      Je l'avais oublié celui-là…
      
      Mais il est bien encore là, voir les occurrences :
      
      2022/11/01 2 2022/11/02 2 2022/11/03 1 2022/11/04 0 2022/11/05 3 2022/11/06 4 2022/11/07 3 2022/11/08 2 2022/11/09 4 2022/11/10 0 2022/11/11 3 2022/11/12 1 2022/11/13 1 2022/11/14 1 2022/11/15 4 2022/11/16 2 2022/11/17 1 2022/11/18 1
      
      -> https://github.com/linuxfrorg/linuxfr.org/pull/346
      Répondre
      - [^] # Re: Creusons un peu
        
        Posté par Benoît Sibaud (site web personnel) le 20 novembre 2022 à 17:55. Évalué à 3 (+0/-0).
        
        Déployé. https://github.com/linuxfrorg/linuxfr.org/commit/f1385d0cec3cdac4ee3edd99f83e3c3d82fcaeb8
        
        Répondre
- [^] # Re: Creusons un peu
  
  Posté par devnewton 🍺 (site web personnel) le 17 novembre 2021 à 11:11. Évalué à 3 (+0/-0).
  
  Les Go-http-client sont peut être les deux instances de gb3 ? https://gb3.devnewton.fr/ et https://gb3.plop.cc/ ?
  
  Si je peux faire quelque-chose côté client…
  
  Le post ci-dessus est une grosse connerie, ne le lisez pas sérieusement.
  
  Répondre

Envoyer un commentaire

Suivre le flux des commentaires

Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.