Project

General

Profile

Bug #22312

ajouter le host dans les logs uwsgi

Added by Thomas Noël almost 2 years ago. Updated 10 months ago.

Status:
Solution proposée
Priority:
Normal
Assignee:
Target version:
-
Start date:
06 Mar 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Logs actuels uwsgi visibles via journalctl:

mars 06 13:46:03 auquo-test uwsgi[13664]: {address space usage: 2899169280 bytes/2764MB} {rss usage: 997912576 bytes/951MB} 
  [pid: 13673|app: 0|req: 18732/18732] 0.0.0.0 () {46 vars in 1154 bytes} [Tue Mar  6 13:46:03 2018] GET /backoffice/workflows/3/status/12/ =>
  generated 42190 bytes in 299 msecs (HTTP/1.0 200) 4 headers in 152 bytes (1 switches on core 18)

Il manque, au moins, le host.

0001-debian-enhance-log-format-in-uwsgi.ini-22312.patch View (1003 Bytes) Thomas Noël, 06 Mar 2018 03:25 PM

History

#1 Updated by Thomas Noël almost 2 years ago

  • Description updated (diff)

#2 Updated by Thomas Noël almost 2 years ago

En suivant http://uwsgi-docs.readthedocs.io/en/latest/LogFormat.html (et en finissant par regarder le code de uwsgi pour tout piger)

J'ajoute le host dans %(host)(uri)

J'en profite pour ajouter le referer, l'agent, le x-forwarded-for (pas trouvé autre chose d'accessible dans uwsgi), les éventuelles ioerror, la conso ram vsz/rss, la taille du body, etc.

Avant:

mars 05 23:52:31 auquo-test uwsgi[13664]: {address space usage: 2891137024 bytes/2757MB} {rss usage: 777093120 bytes/741MB}
 [pid: 13673|app: 0|req: 4306/4306] 0.0.0.0 () {40 vars in 605 bytes}
 [Mon Mar  5 23:52:31 2018] GET /__metrics__/ => generated 101922 bytes in 23 msecs (HTTP/1.0 200)
 1 headers in 75 bytes (1 switches on core 1)

Après :

mars 06 15:18:31 auquo-test uwsgi[18284]:
  [pid:18294|worker:1] 0.0.0.0,145.239.186.85 (user:-) {40 vars in 605 bytes, body:0 bytes}
  [Tue Mar  6 15:18:31 2018] GET demarches-meyzieu.test.au-quotidien.com:443/__metrics__/ => 87158 bytes in 23 msecs (HTTP/1.0 200)
  1 headers in 75 bytes (1 switches on core 0 vsz:2695M rss:463M) (0 errors read:0 write:0)
  referer:- agent:Prometheus/2.1.0

PS: On peut peut-être rendre le fonctionnel via du hack dans nginx, ça pourrait éviter le %(addr),(var.HTTP_X_FORWARDED_FOR) (je crois que c'est possible mais je ne retrouve plus comment...)

#3 Updated by Frédéric Péters 10 months ago

  • Status changed from Nouveau to Solution proposée

On a encore envie de faire ça ? (sur l'idée qu'on récemment retiré les infos "mémoire") (mais peut-être qu'on veut quand même ajouter le host) (ou on se dit qu'on doit avoir des logs corrects côté haproxy/nginx ?).

#4 Updated by Thomas Noël 10 months ago

Dans le faits, je ne suis jamais allé voir les logs uwsgi pour déboguer un truc, je suis toujours passé par nginx quand il fallait aller voir la couche http. Donc je me dis que ce ticket peut être rejeté/annulé, sans intérêt. Sauf si qqun dit qu'il utilise les logs uwsgi de son côté ?...

Also available in: Atom PDF