Merge pull request #258 from mclazarus/master

Don't quote shell escaped values.
This commit is contained in:
David Dollar committed 2013-01-14 05:10:36 -08:00
commit 88cdaacc67
4 files changed
+6 -6

No files matched your search

+1 -1
View File
@@ -91,7 +91,7 @@ private ######################################################################
end end
def shell_quote(value) def shell_quote(value)
'"' + Shellwords.escape(value) + '"' Shellwords.escape(value)
end end
# deprecated # deprecated
+1 -1
View File
@@ -38,7 +38,7 @@ describe Foreman::Export::Upstart, :fakefs do
engine.env['KEY'] = 'd"\|d' engine.env['KEY'] = 'd"\|d'
upstart.export upstart.export
"foobarfoo".should include "bar" "foobarfoo".should include "bar"
File.read("/tmp/init/app-alpha-1.conf").should =~ /KEY="d\\"\\\\\\\|d/ File.read("/tmp/init/app-alpha-1.conf").should =~ /KEY=d\\"\\\\\\\|d/
end end
context "with a formation" do context "with a formation" do
@@ -8,7 +8,7 @@ stdout_logfile=/var/log/app/alpha-1.log
stderr_logfile=/var/log/app/alpha-1.error.log stderr_logfile=/var/log/app/alpha-1.error.log
user=app user=app
directory=/tmp/app directory=/tmp/app
environment=PORT="5000" environment=PORT=5000
[program:app-bravo-1] [program:app-bravo-1]
command=./bravo command=./bravo
autostart=true autostart=true
@@ -18,7 +18,7 @@ stdout_logfile=/var/log/app/bravo-1.log
stderr_logfile=/var/log/app/bravo-1.error.log stderr_logfile=/var/log/app/bravo-1.error.log
user=app user=app
directory=/tmp/app directory=/tmp/app
environment=PORT="5100" environment=PORT=5100
[group:app] [group:app]
programs=app-alpha-1,app-bravo-1 programs=app-alpha-1,app-bravo-1
@@ -8,7 +8,7 @@ stdout_logfile=/var/log/app/alpha-1.log
stderr_logfile=/var/log/app/alpha-1.error.log stderr_logfile=/var/log/app/alpha-1.error.log
user=app user=app
directory=/tmp/app directory=/tmp/app
environment=PORT="5000" environment=PORT=5000
[program:app-alpha-2] [program:app-alpha-2]
command=./alpha command=./alpha
autostart=true autostart=true
@@ -18,7 +18,7 @@ stdout_logfile=/var/log/app/alpha-2.log
stderr_logfile=/var/log/app/alpha-2.error.log stderr_logfile=/var/log/app/alpha-2.error.log
user=app user=app
directory=/tmp/app directory=/tmp/app
environment=PORT="5001" environment=PORT=5001
[group:app] [group:app]
programs=app-alpha-1,app-alpha-2 programs=app-alpha-1,app-alpha-2