diff options
author | Zhidao HONG <z.hong@f5.com> | 2022-09-28 15:51:55 +0100 |
---|---|---|
committer | Andrew Clayton <andrew@digital-domain.net> | 2022-10-04 19:33:11 +0100 |
commit | dc9f592d6e7123d57924146dcbf1be80366bc98b (patch) | |
tree | 372c7377a1420705a1a39acfdd12d126136a9307 /src/nxt_string.h | |
parent | b00983369be5f356280168b4c5d600bd7d614c60 (diff) | |
download | unit-dc9f592d6e7123d57924146dcbf1be80366bc98b.tar.gz unit-dc9f592d6e7123d57924146dcbf1be80366bc98b.tar.bz2 |
Ruby: added support for rack V3.
Ruby applications would fail to start if they were using rack v3
2022/09/28 15:48:46 [alert] 0#80912 [unit] Ruby: Failed to parse rack script
2022/09/28 15:48:46 [notice] 80911#80911 app process 80912 exited with code 1
This was due to a change in the rack API
Rack V2
def self.load_file(path, opts = Server::Options.new)
...
cfgfile.sub!(/^__END__\n.*\Z/m, '')
app = new_from_string cfgfile, path
return app, options
end
Rack V3
def self.load_file(path)
...
return new_from_string(config, path)
end
This patch handles _both_ the above APIs by correctly handling the cases
where we do and don't get an array returned from
nxt_ruby_rack_parse_script().
Closes: <https://github.com/nginx/unit/issues/755>
Tested-by: Andrew Clayton <a.clayton@nginx.com>
Reviewed-by: Andrew Clayton <a.clayton@nginx.com>
[ Andrew: Patch by Zhidao, commit message by me with input from Zhidao ]
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to 'src/nxt_string.h')
0 files changed, 0 insertions, 0 deletions