Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 9422170: Run Discourse via Thin rather than FGCI (Closed)

Created:
Feb. 27, 2013, 1:34 p.m. by Wladimir Palant
Modified:
Nov. 8, 2013, 8:06 a.m.
Reviewers:
Felix Dahlke
Visibility:
Public.

Description

Run Discourse via Thin rather than FGCI

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -67 lines) Patch
R modules/discourse/files/discourse.fcgi View 1 chunk +0 lines, -27 lines 0 comments Download
M modules/discourse/files/init-discourse View 2 chunks +5 lines, -14 lines 0 comments Download
M modules/discourse/files/intraforum.adblockplus.org View 2 chunks +8 lines, -2 lines 1 comment Download
M modules/discourse/files/sudoers View 1 chunk +2 lines, -2 lines 0 comments Download
M modules/discourse/manifests/init.pp View 3 chunks +5 lines, -22 lines 0 comments Download

Messages

Total messages: 2
Wladimir Palant
Feb. 27, 2013, 1:34 p.m. (2013-02-27 13:34:09 UTC) #1
Felix Dahlke
Feb. 27, 2013, 1:38 p.m. (2013-02-27 13:38:06 UTC) #2
LGTM

http://codereview.adblockplus.org/9422170/diff/1/modules/discourse/files/intr...
File modules/discourse/files/intraforum.adblockplus.org (right):

http://codereview.adblockplus.org/9422170/diff/1/modules/discourse/files/intr...
modules/discourse/files/intraforum.adblockplus.org:47: proxy_set_header
X-Real-IP  $remote_addr;
How about just one space between P and $?

Powered by Google App Engine
This is Rietveld