Petites correction après migration de serveur

[ymph]

Bonjour,

J'ai fait 2 petite corrections suite à une migration de serveur et l'installation de la version v2.6.4 de co-ment.

• dans le template tracking.html: j'ai pris l'option de supprimer le bout de code qui semblait en trop.
• dans activity.py: J'ai déployé une solution nginx/gunicorn avec l'utilisation de proxy. dans ce acas il semble que le header 'REMOTE_ADDR' est défini mais contient la chaine vide. ce qui créait ensuite une erreur dans la base de donnée lors de l'insertion de l'activité.
  De plus dans Nginx, j'ai la configuration suivante (assez classique finalement) pour la propagation des header vers gunicorn par le proxy : proxy_set_header X-Real-IP $remote_addr;
  D'où ma correction...

Je les partage avec vous afin de voir si cela fait sens pour vous de les récupérer.

Cordialement,

Yves-Marie Haussonne

[sfermigier]

Super, on regarde ça dans la semaine.

2016-03-29 14:53 GMT+02:00 Haussonne Yves-Marie [email protected]:

Bonjour,

J'ai fait 2 petite corrections suite à une migration de serveur et
l'installation de la version v2.6.4 de co-ment.

• dans le template tracking.html: j'ai pris l'option de supprimer le
  bout de code qui semblait en trop.
• dans activity.py: J'ai déployé une solution nginx/gunicorn avec
  l'utilisation de proxy. dans ce acas il semble que le header 'REMOTE_ADDR'
  est défini mais contient la chaine vide. ce qui créait ensuite une erreur
  dans la base de donnée lors de l'insertion de l'activité. De plus dans
  Nginx, j'ai la configuration suivante (assez classique finalement) pour la
  propagation des header vers gunicorn par le proxy : proxy_set_header
  X-Real-IP $remote_addr; D'où ma correction...

Je les partage avec vous afin de voir si cela fait sens pour vous de les
récupérer.

Cordialement,

Yves-Marie Haussonne

You can view, comment on, or merge this pull request online at:

#6
Commit Summary

• Correct tracking template. Either there is a script tag and part of
  code missing, either this is a left-over that need to be deleted.
• small change to adress the case when the REMOTE_ADDR header is empty
  or when using a proxy

File Changes

• M src/cm/activity.py
  https://github.com/co-ment/comt/pull/6/files#diff-0 (28)
• M src/cm/templates/site/tracking.html
  https://github.com/co-ment/comt/pull/6/files#diff-1 (7)

Patch Links:

• https://github.com/co-ment/comt/pull/6.patch
• https://github.com/co-ment/comt/pull/6.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#6

Stefane Fermigier - http://fermigier.com/ - http://twitter.com/sfermigier -
http://linkedin.com/in/sfermigier
Founder & CEO, Abilian - Enterprise Social Software -
http://www.abilian.com/
Founder, Nuxeo - Enterprise Content Management Platform -
http://www.nuxeo.com/
Chairman, Free&OSS Group / Systematic Cluster -
http://www.gt-logiciel-libre.org/
Co-President, National Council for Free & Open Source Software (CNLL) -
http://cnll.fr/

Founder & Organiser, PyData Paris - http://pydataparis.joinux.org/

"Le vrai courage c'est de faire ce qui est juste" - Docteur Benjamin Justice