unicorn 5.4.0 → 5.5.0

Sign up to get free protection for your applications and to get access to all the features.
Files changed (47) hide show
  1. checksums.yaml +4 -4
  2. data/.manifest +2 -0
  3. data/.olddoc.yml +1 -1
  4. data/Application_Timeouts +4 -4
  5. data/Documentation/unicorn.1.txt +1 -1
  6. data/Documentation/unicorn_rails.1.txt +1 -1
  7. data/GIT-VERSION-FILE +1 -1
  8. data/GIT-VERSION-GEN +1 -1
  9. data/ISSUES +5 -2
  10. data/LATEST +25 -8
  11. data/LICENSE +2 -2
  12. data/Links +9 -7
  13. data/NEWS +71 -0
  14. data/README +13 -6
  15. data/Sandbox +2 -2
  16. data/bin/unicorn +3 -1
  17. data/examples/logrotate.conf +1 -1
  18. data/examples/nginx.conf +3 -2
  19. data/ext/unicorn_http/common_field_optimization.h +24 -6
  20. data/ext/unicorn_http/extconf.rb +30 -0
  21. data/ext/unicorn_http/global_variables.h +2 -2
  22. data/ext/unicorn_http/httpdate.c +2 -2
  23. data/ext/unicorn_http/unicorn_http.c +4 -9
  24. data/ext/unicorn_http/unicorn_http.rl +4 -9
  25. data/lib/unicorn.rb +19 -8
  26. data/lib/unicorn/configurator.rb +12 -1
  27. data/lib/unicorn/http_request.rb +1 -2
  28. data/lib/unicorn/http_server.rb +19 -20
  29. data/lib/unicorn/launcher.rb +1 -1
  30. data/lib/unicorn/socket_helper.rb +3 -2
  31. data/lib/unicorn/util.rb +3 -3
  32. data/lib/unicorn/version.rb +1 -1
  33. data/lib/unicorn/worker.rb +16 -2
  34. data/man/man1/unicorn.1 +7 -5
  35. data/man/man1/unicorn_rails.1 +6 -3
  36. data/t/README +4 -4
  37. data/t/t0301-no-default-middleware-ignored-in-config.sh +25 -0
  38. data/t/t0301.ru +13 -0
  39. data/test/exec/test_exec.rb +6 -7
  40. data/test/unit/test_ccc.rb +1 -1
  41. data/test/unit/test_http_parser.rb +16 -0
  42. data/test/unit/test_server.rb +5 -5
  43. data/test/unit/test_signals.rb +2 -2
  44. data/test/unit/test_socket_helper.rb +4 -4
  45. data/test/unit/test_util.rb +25 -0
  46. data/unicorn.gemspec +1 -1
  47. metadata +5 -4
checksums.yaml CHANGED
@@ -1,7 +1,7 @@
1
1
  ---
2
2
  SHA256:
3
- metadata.gz: 2a53c6f0287f74563b46ac07e96cbb709562e4f26bdbf46ae1cb904a3e345828
4
- data.tar.gz: 0724bceedd3b964168081c4a12c8f9eb04611013f6ef72593d6db63bd31cccf1
3
+ metadata.gz: c37db62d1e413297803b2fac9be18b752b80b0068eeeafd1027eb2eae2a693c2
4
+ data.tar.gz: 0af88fc316857d8e0c4bc899de3d7994f54a01e5a9a8a25ad5a721cc669d6bf9
5
5
  SHA512:
6
- metadata.gz: a0b91ca7dd6cc2e52f7eed8fdd30547dbcd3c81939c1df495e612ff94a544dc589cd3d3cd9e4c17206f135ac7e3ef763ecfa80bebacce42960df7f93a679d091
7
- data.tar.gz: 9eba1d1fefa858d93524ff73c77b8a239b9e50abb6f8bac840d436afebd27608dcad6a8d7f49c35dd44f2577563fdbdc898541efa312b7f5953c155cf1eaf057
6
+ metadata.gz: 5788c3a37e65de0dbc12735f72af22a27638ada89f05c3e2904171662fd72ba3ef42d01efd3aa3bd8c3324a90c2b3b11fbd27835b7c8f32969b923a66dae9e65
7
+ data.tar.gz: 19928922c0e6068b4818d2111f9f0e47ffe071c22c68deb23c67a29c787c9e0f5e1e17d503c56b2e01f885aa9329ca04e5e7829ccf28e3d7cec14907f9b20736
data/.manifest CHANGED
@@ -128,6 +128,8 @@ t/t0116-client_body_buffer_size.sh
128
128
  t/t0116.ru
129
129
  t/t0200-rack-hijack.sh
130
130
  t/t0300-no-default-middleware.sh
131
+ t/t0301-no-default-middleware-ignored-in-config.sh
132
+ t/t0301.ru
131
133
  t/t9000-preread-input.sh
132
134
  t/t9001-oob_gc.sh
133
135
  t/t9002-oob_gc-path.sh
data/.olddoc.yml CHANGED
@@ -1,6 +1,6 @@
1
1
  ---
2
2
  cgit_url: https://bogomips.org/unicorn.git
3
- git_url: git://bogomips.org/unicorn.git
3
+ git_url: https://bogomips.org/unicorn.git
4
4
  rdoc_url: https://bogomips.org/unicorn/
5
5
  ml_url: https://bogomips.org/unicorn-public/
6
6
  merge_html:
data/Application_Timeouts CHANGED
@@ -23,10 +23,10 @@ Most database adapters allow configurable timeouts.
23
23
  Net::HTTP and Net::SMTP in the Ruby standard library allow
24
24
  configurable timeouts.
25
25
 
26
- Even for things as fast as {memcached}[http://memcached.org/],
27
- {dalli}[http://rubygems.org/gems/dalli],
28
- {memcached}[http://rubygems.org/gems/memcached] and
29
- {memcache-client}[http://rubygems.org/gems/memcache-client] RubyGems all
26
+ Even for things as fast as {memcached}[https://memcached.org/],
27
+ {dalli}[https://rubygems.org/gems/dalli],
28
+ {memcached}[https://rubygems.org/gems/memcached] and
29
+ {memcache-client}[https://rubygems.org/gems/memcache-client] RubyGems all
30
30
  offer configurable timeouts.
31
31
 
32
32
  Consult the relevant documentation for the libraries you use on
@@ -182,6 +182,6 @@ the unicorn config file.
182
182
  * [Rackup HowTo][3]
183
183
 
184
184
  [1]: https://bogomips.org/unicorn/
185
- [2]: http://www.rubydoc.info/github/rack/rack/
185
+ [2]: https://www.rubydoc.info/github/rack/rack/
186
186
  [3]: https://github.com/rack/rack/wiki/tutorial-rackup-howto
187
187
  [4]: https://bogomips.org/unicorn/SIGNALS.html
@@ -170,6 +170,6 @@ used by Unicorn.
170
170
  * [Rackup HowTo][3]
171
171
 
172
172
  [1]: https://bogomips.org/unicorn/
173
- [2]: http://www.rubydoc.info/github/rack/rack/
173
+ [2]: https://www.rubydoc.info/github/rack/rack/
174
174
  [3]: https://github.com/rack/rack/wiki/tutorial-rackup-howto
175
175
  [4]: https://bogomips.org/unicorn/SIGNALS.html
data/GIT-VERSION-FILE CHANGED
@@ -1 +1 @@
1
- GIT_VERSION = 5.4.0
1
+ GIT_VERSION = 5.5.0
data/GIT-VERSION-GEN CHANGED
@@ -1,5 +1,5 @@
1
1
  #!/usr/bin/env ruby
2
- DEF_VER = "v5.4.0"
2
+ DEF_VER = "v5.5.0"
3
3
  CONSTANT = "Unicorn::Const::UNICORN_VERSION"
4
4
  RVF = "lib/unicorn/version.rb"
5
5
  GVF = "GIT-VERSION-FILE"
data/ISSUES CHANGED
@@ -39,6 +39,7 @@ https://bugs.ruby-lang.org/ and discuss+fix them on the ruby-core
39
39
  list at mailto:ruby-core@ruby-lang.org
40
40
  Subscription to post is required to ruby-core, unfortunately:
41
41
  mailto:ruby-core-request@ruby-lang.org?subject=subscribe
42
+ Unofficial archives are available at: https://public-inbox.org/ruby-core/
42
43
 
43
44
  For uncommon bugs in Rack, we may forward bugs to
44
45
  mailto:rack-devel@googlegroups.com and discuss there.
@@ -46,6 +47,7 @@ Subscription (without any web UI or Google account) is possible via:
46
47
  mailto:rack-devel+subscribe@googlegroups.com
47
48
  Note: not everyone can use the proprietary bug tracker used by Rack,
48
49
  but their mailing list remains operational.
50
+ Unofficial archives are available at: https://public-inbox.org/rack-devel/
49
51
 
50
52
  Uncommon bugs we encounter in the Linux kernel should be Cc:-ed to the
51
53
  Linux kernel mailing list (LKML) at mailto:linux-kernel@vger.kernel.org
@@ -54,11 +56,12 @@ and subsystem maintainers such as mailto:netdev@vger.kernel.org
54
56
  involved with any problematic commits (including those in the
55
57
  Signed-off-by: and other trailer lines). No subscription is necessary,
56
58
  and the our mailing list follows the same conventions as LKML for
57
- interopability. There is a kernel.org Bugzilla instance, but it is
58
- ignored by most developers.
59
+ interopability. Archives are available at https://lore.kernel.org/lkml/
60
+ There is a kernel.org Bugzilla instance, but it is ignored by most.
59
61
 
60
62
  Likewise for any rare glibc bugs we might encounter, we should Cc:
61
63
  mailto:libc-alpha@sourceware.org
64
+ Unofficial archives are available at: https://public-inbox.org/libc-alpha/
62
65
  Keep in mind glibc upstream does use Bugzilla for tracking bugs:
63
66
  https://sourceware.org/bugzilla/
64
67
 
data/LATEST CHANGED
@@ -1,13 +1,30 @@
1
- === unicorn 5.3.1 / 2017-10-03 19:03 UTC
1
+ === unicorn 5.5.0.pre1 / 2018-12-20 20:11 UTC
2
2
 
3
- This release fixes an occasional GC problem introduced in v5.3.0
4
- to reduce global variable overhead (commit 979ebcf91705709b)
3
+ Jeremy Evans contributed the "default_middleware" configuration option:
5
4
 
6
- Thanks to Xuanzhong Wei for the patch which lead to this release:
5
+ https://bogomips.org/unicorn-public/20180913192055.GD48926@jeremyevans.local/
7
6
 
8
- https://bogomips.org/unicorn-public/20171003182054.76392-1-azrlew@gmail.com/T/#u
9
- https://bogomips.org/unicorn-public/20171003145718.30404-1-azrlew@gmail.com/T/#u
7
+ Jeremy also contributed the ability to use separate groups for the process
8
+ and log files:
10
9
 
11
- Xuanzhong Wei (1):
12
- fix GC issue on rb_global_variable array
10
+ https://bogomips.org/unicorn-public/20180913192449.GE48926@jeremyevans.local/
11
+
12
+ There's also a couple of uninteresting minor optimizations and
13
+ documentation additions.
14
+
15
+ Eric Wong (10):
16
+ remove random seed reset atfork
17
+ use IO#wait instead of kgio_wait_readable
18
+ Merge branch '5.4-stable'
19
+ shrink pipes under Linux
20
+ socket_helper: add hint for FreeBSD users for accf_http(9)
21
+ tests: ensure -N/--no-default-middleware not supported in config.ru
22
+ doc: update more URLs to use HTTPS and avoid redirects
23
+ deduplicate strings VM-wide in Ruby 2.5+
24
+ doc/ISSUES: add links to git clone-able mail archives of our dependencies
25
+ README: minor updates and additional disclaimer
26
+
27
+ Jeremy Evans (2):
28
+ Make Worker#user support different process primary group and log file group
29
+ Support default_middleware configuration option
13
30
 
data/LICENSE CHANGED
@@ -8,8 +8,8 @@ any later version. We currently prefer the GPLv3 or later for
8
8
  derivative works, but the GPLv2 is fine.
9
9
 
10
10
  The complete texts of the GPLv2 and GPLv3 are below:
11
- GPLv2 - http://www.gnu.org/licenses/gpl-2.0.txt
12
- GPLv3 - http://www.gnu.org/licenses/gpl-3.0.txt
11
+ GPLv2 - https://www.gnu.org/licenses/gpl-2.0.txt
12
+ GPLv3 - https://www.gnu.org/licenses/gpl-3.0.txt
13
13
 
14
14
  You may (against our _preference_) also use the Ruby 1.8 license terms
15
15
  which we inherited from the original Mongrel project when we forked it:
data/Links CHANGED
@@ -10,7 +10,7 @@ The unicorn project is not responsible for the content in these links.
10
10
  Furthermore, the unicorn project has never, does not and will never endorse:
11
11
 
12
12
  * any for-profit entities or services
13
- * any non-{Free Software}[http://www.gnu.org/philosophy/free-sw.html]
13
+ * any non-{Free Software}[https://www.gnu.org/philosophy/free-sw.html]
14
14
 
15
15
  The existence of these links does not imply endorsement of any entities
16
16
  or services behind them.
@@ -31,26 +31,28 @@ or services behind them.
31
31
 
32
32
  === unicorn is written to work with
33
33
 
34
- * {Rack}[http://rack.github.io/] - a minimal interface between webservers
34
+ * {Rack}[https://rack.github.io/] - a minimal interface between webservers
35
35
  supporting Ruby and Ruby frameworks
36
36
 
37
37
  * {Ruby}[https://www.ruby-lang.org/en/] - the programming language of
38
38
  Rack and unicorn
39
39
 
40
- * {nginx}[http://nginx.org/] (Free versions) -
40
+ * {nginx}[https://nginx.org/] (Free versions) -
41
41
  the reverse proxy for use with unicorn
42
42
 
43
43
  === Derivatives
44
44
 
45
- * {Green Unicorn}[http://gunicorn.org/] - a Python version of unicorn
45
+ * {Green Unicorn}[https://gunicorn.org/] - a Python version of unicorn
46
46
 
47
- * {Starman}[http://search.cpan.org/dist/Starman/] - Plack/PSGI version
47
+ * {Starman}[https://metacpan.org/release/Starman/] - Plack/PSGI version
48
48
  of unicorn
49
49
 
50
50
  === Prior Work
51
51
 
52
- * {Mongrel}[http://rubygems.org/gems/mongrel] - the awesome webserver
53
- unicorn is based on
52
+ * {Mongrel}[https://rubygems.org/gems/mongrel] - the awesome webserver
53
+ unicorn is based on. A historical archive of the mongrel dev list
54
+ featuring early discussions of unicorn is available at:
55
+ https://bogomips.org/mongrel-devel/
54
56
 
55
57
  * {david}[https://bogomips.org/david.git] - a tool to explain why you need
56
58
  nginx in front of unicorn
data/NEWS CHANGED
@@ -1,3 +1,74 @@
1
+ === unicorn 5.5.0.pre1 / 2018-12-20 20:11 UTC
2
+
3
+ Jeremy Evans contributed the "default_middleware" configuration option:
4
+
5
+ https://bogomips.org/unicorn-public/20180913192055.GD48926@jeremyevans.local/
6
+
7
+ Jeremy also contributed the ability to use separate groups for the process
8
+ and log files:
9
+
10
+ https://bogomips.org/unicorn-public/20180913192449.GE48926@jeremyevans.local/
11
+
12
+ There's also a couple of uninteresting minor optimizations and
13
+ documentation additions.
14
+
15
+ Eric Wong (10):
16
+ remove random seed reset atfork
17
+ use IO#wait instead of kgio_wait_readable
18
+ Merge branch '5.4-stable'
19
+ shrink pipes under Linux
20
+ socket_helper: add hint for FreeBSD users for accf_http(9)
21
+ tests: ensure -N/--no-default-middleware not supported in config.ru
22
+ doc: update more URLs to use HTTPS and avoid redirects
23
+ deduplicate strings VM-wide in Ruby 2.5+
24
+ doc/ISSUES: add links to git clone-able mail archives of our dependencies
25
+ README: minor updates and additional disclaimer
26
+
27
+ Jeremy Evans (2):
28
+ Make Worker#user support different process primary group and log file group
29
+ Support default_middleware configuration option
30
+
31
+ === unicorn 5.4.1 / 2018-07-23 17:13 UTC
32
+
33
+ This release quiets some warnings for Ruby 2.6 preview releases
34
+ and enables tests to pass under Ruby 1.9.3. Otherwise, nothing
35
+ interesting for Ruby 2.0..2.5 users. *YAWN*
36
+
37
+ Eric Wong (1):
38
+ quiet some mismatched indentation warnings
39
+
40
+ Fumiaki MATSUSHIMA (1):
41
+ Use IO#wait instead to fix test for Ruby 1.9
42
+
43
+ === unicorn 5.4.0 / 2017-12-23 23:33 UTC
44
+
45
+ Rack hijack support improves as the app code can capture and use
46
+ the Rack `env' privately without copying it (to avoid clobbering
47
+ by another client). Thanks to Sam Saffron for reporting and
48
+ testing this new feature:
49
+ https://bogomips.org/unicorn-public/CAAtdryPG3nLuyo0jxfYW1YHu1Q+ZpkLkd4KdWC8vA46B5haZxw@mail.gmail.com/T/
50
+
51
+ We also now support $DEBUG being set by the Rack app (instead of
52
+ relying on the "-d" CLI switch). Thanks to James P Robinson Jr
53
+ for reporting this bug:
54
+ https://bogomips.org/unicorn-public/D6324CB4.7BC3E%25james.robinson3@cigna.com/T/
55
+ (Coincidentally, this fix will be irrelevant for Ruby 2.5
56
+ which requires 'pp' by default)
57
+
58
+ There's a few minor test cleanups and documentation updates, too.
59
+
60
+ All commits since v5.3.1 (2017-10-03):
61
+
62
+ reduce method calls with String#start_with?
63
+ require 'pp' if $DEBUG is set by Rack app
64
+ avoid reusing env on hijack
65
+ tests: cleanup some unused variable warnings
66
+ ISSUES: add a note about Debian BTS interopability
67
+
68
+ Roughly all mailing discussions since the last release:
69
+
70
+ https://bogomips.org/unicorn-public/?q=d:20171004..20171223
71
+
1
72
  === unicorn 5.3.1 / 2017-10-03 19:03 UTC
2
73
 
3
74
  This release fixes an occasional GC problem introduced in v5.3.0
data/README CHANGED
@@ -10,7 +10,7 @@ both the the request and response in between unicorn and slow clients.
10
10
 
11
11
  * Designed for Rack, Unix, fast clients, and ease-of-debugging. We
12
12
  cut out everything that is better supported by the operating system,
13
- {nginx}[http://nginx.org/] or {Rack}[http://rack.github.io/].
13
+ {nginx}[https://nginx.org/] or {Rack}[https://rack.github.io/].
14
14
 
15
15
  * Compatible with Ruby 1.9.3 and later.
16
16
  unicorn 4.x remains supported for Ruby 1.8 users.
@@ -37,12 +37,15 @@ both the the request and response in between unicorn and slow clients.
37
37
  You can upgrade unicorn, your entire application, libraries
38
38
  and even your Ruby interpreter without dropping clients.
39
39
 
40
+ * transparent upgrades using systemd socket activation is
41
+ supported since unicorn 5.0
42
+
40
43
  * before_fork and after_fork hooks in case your application
41
44
  has special needs when dealing with forked processes. These
42
45
  should not be needed when the "preload_app" directive is
43
46
  false (the default).
44
47
 
45
- * Can be used with copy-on-write-friendly memory management
48
+ * Can be used with copy-on-write-friendly GC in Ruby 2.0+
46
49
  to save memory (by setting "preload_app" to true).
47
50
 
48
51
  * Able to listen on multiple interfaces including UNIX sockets,
@@ -55,7 +58,7 @@ both the the request and response in between unicorn and slow clients.
55
58
 
56
59
  == License
57
60
 
58
- unicorn is copyright 2009-2016 by all contributors (see logs in git).
61
+ unicorn is copyright 2009-2018 by all contributors (see logs in git).
59
62
  It is based on Mongrel 1.1.5.
60
63
  Mongrel is copyright 2007 Zed A. Shaw and contributors.
61
64
 
@@ -77,13 +80,13 @@ You may install it via RubyGems on RubyGems.org:
77
80
  You can get the latest source via git from the following locations
78
81
  (these versions may not be stable):
79
82
 
80
- git://bogomips.org/unicorn.git
81
- git://repo.or.cz/unicorn.git (mirror)
83
+ https://bogomips.org/unicorn.git
84
+ https://repo.or.cz/unicorn.git (mirror)
82
85
 
83
86
  You may browse the code from the web:
84
87
 
85
88
  * https://bogomips.org/unicorn.git
86
- * http://repo.or.cz/w/unicorn.git (gitweb)
89
+ * https://repo.or.cz/w/unicorn.git (gitweb)
87
90
 
88
91
  See the HACKING guide on how to contribute and build prerelease gems
89
92
  from git.
@@ -122,6 +125,10 @@ unicorn is designed to only serve fast clients either on the local host
122
125
  or a fast LAN. See the PHILOSOPHY and DESIGN documents for more details
123
126
  regarding this.
124
127
 
128
+ Due to its ability to tolerate crashes and isolate clients, unicorn
129
+ is unfortunately known to prolong the existence of bugs in applications
130
+ and libraries which run on top of it.
131
+
125
132
  == Contact
126
133
 
127
134
  All feedback (bug reports, user/development dicussion, patches, pull
data/Sandbox CHANGED
@@ -3,7 +3,7 @@
3
3
  Since unicorn includes executables and is usually used to start a Ruby
4
4
  process, there are certain caveats to using it with tools that sandbox
5
5
  RubyGems installations such as
6
- {Bundler}[http://bundler.io/] or
6
+ {Bundler}[https://bundler.io/] or
7
7
  {Isolate}[https://github.com/jbarnette/isolate].
8
8
 
9
9
  == General deployment
@@ -66,7 +66,7 @@ before_exec hook as illustrated by https://gist.github.com/534668
66
66
  Ruby 2.0.0 enforces FD_CLOEXEC on file descriptors by default. unicorn
67
67
  has been prepared for this behavior since unicorn 4.1.0, and bundler
68
68
  needs the "--keep-file-descriptors" option for "bundle exec":
69
- http://bundler.io/man/bundle-exec.1.html
69
+ https://bundler.io/man/bundle-exec.1.html
70
70
 
71
71
  == Isolate
72
72
 
data/bin/unicorn CHANGED
@@ -6,6 +6,7 @@
6
6
  ENV["RACK_ENV"] ||= "development"
7
7
  rackup_opts = Unicorn::Configurator::RACKUP
8
8
  options = rackup_opts[:options]
9
+ set_no_default_middleware = true
9
10
 
10
11
  op = OptionParser.new("", 24, ' ') do |opts|
11
12
  cmd = File.basename($0)
@@ -60,7 +61,7 @@
60
61
 
61
62
  opts.on("-N", "--no-default-middleware",
62
63
  "do not load middleware implied by RACK_ENV") do |e|
63
- rackup_opts[:no_default_middleware] = true
64
+ rackup_opts[:no_default_middleware] = true if set_no_default_middleware
64
65
  end
65
66
 
66
67
  opts.on("-D", "--daemonize", "run daemonized in the background") do |d|
@@ -110,6 +111,7 @@
110
111
  opts.parse! ARGV
111
112
  end
112
113
 
114
+ set_no_default_middleware = false
113
115
  app = Unicorn.builder(ARGV[0] || 'config.ru', op)
114
116
  op = nil
115
117
 
@@ -2,7 +2,7 @@
2
2
  # /etc/logrotate.d/unicorn_app on my Debian systems
3
3
  #
4
4
  # See the logrotate(8) manpage for more information:
5
- # http://linux.die.net/man/8/logrotate
5
+ # https://linux.die.net/man/8/logrotate
6
6
  #
7
7
  # public logrotate-related discussion in our archives:
8
8
  # https://bogomips.org/unicorn-public/?q=logrotate
data/examples/nginx.conf CHANGED
@@ -56,7 +56,8 @@ http {
56
56
  # to configure it all in one place here for static files and also
57
57
  # to disable gzip for clients who don't get gzip/deflate right.
58
58
  # There are other gzip settings that may be needed used to deal with
59
- # bad clients out there, see http://wiki.nginx.org/NginxHttpGzipModule
59
+ # bad clients out there, see
60
+ # https://nginx.org/en/docs/http/ngx_http_gzip_module.html
60
61
  gzip on;
61
62
  gzip_http_version 1.0;
62
63
  gzip_proxied any;
@@ -117,7 +118,7 @@ http {
117
118
 
118
119
  location @app {
119
120
  # an HTTP header important enough to have its own Wikipedia entry:
120
- # http://en.wikipedia.org/wiki/X-Forwarded-For
121
+ # https://en.wikipedia.org/wiki/X-Forwarded-For
121
122
  proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
122
123
 
123
124
  # enable this if you forward HTTPS traffic to unicorn,
@@ -58,26 +58,44 @@ static struct common_field common_http_fields[] = {
58
58
 
59
59
  #define HTTP_PREFIX "HTTP_"
60
60
  #define HTTP_PREFIX_LEN (sizeof(HTTP_PREFIX) - 1)
61
+ static ID id_uminus;
62
+
63
+ /* this dedupes under Ruby 2.5+ (December 2017) */
64
+ static VALUE str_dd_freeze(VALUE str)
65
+ {
66
+ if (STR_UMINUS_DEDUPE)
67
+ return rb_funcall(str, id_uminus, 0);
68
+
69
+ /* freeze,since it speeds up older MRI slightly */
70
+ OBJ_FREEZE(str);
71
+ return str;
72
+ }
73
+
74
+ static VALUE str_new_dd_freeze(const char *ptr, long len)
75
+ {
76
+ return str_dd_freeze(rb_str_new(ptr, len));
77
+ }
61
78
 
62
79
  /* this function is not performance-critical, called only at load time */
63
- static void init_common_fields(VALUE mark_ary)
80
+ static void init_common_fields(void)
64
81
  {
65
82
  int i;
66
83
  struct common_field *cf = common_http_fields;
67
84
  char tmp[64];
85
+
86
+ id_uminus = rb_intern("-@");
68
87
  memcpy(tmp, HTTP_PREFIX, HTTP_PREFIX_LEN);
69
88
 
70
89
  for(i = ARRAY_SIZE(common_http_fields); --i >= 0; cf++) {
71
90
  /* Rack doesn't like certain headers prefixed with "HTTP_" */
72
91
  if (!strcmp("CONTENT_LENGTH", cf->name) ||
73
92
  !strcmp("CONTENT_TYPE", cf->name)) {
74
- cf->value = rb_str_new(cf->name, cf->len);
93
+ cf->value = str_new_dd_freeze(cf->name, cf->len);
75
94
  } else {
76
95
  memcpy(tmp + HTTP_PREFIX_LEN, cf->name, cf->len + 1);
77
- cf->value = rb_str_new(tmp, HTTP_PREFIX_LEN + cf->len);
96
+ cf->value = str_new_dd_freeze(tmp, HTTP_PREFIX_LEN + cf->len);
78
97
  }
79
- cf->value = rb_obj_freeze(cf->value);
80
- rb_ary_push(mark_ary, cf->value);
98
+ rb_gc_register_mark_object(cf->value);
81
99
  }
82
100
  }
83
101
 
@@ -105,7 +123,7 @@ static VALUE uncommon_field(const char *field, size_t flen)
105
123
  memcpy(RSTRING_PTR(f) + HTTP_PREFIX_LEN, field, flen);
106
124
  assert(*(RSTRING_PTR(f) + RSTRING_LEN(f)) == '\0' &&
107
125
  "string didn't end with \\0"); /* paranoia */
108
- return rb_obj_freeze(f);
126
+ return HASH_ASET_DEDUPE ? f : str_dd_freeze(f);
109
127
  }
110
128
 
111
129
  #endif /* common_field_optimization_h */