Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add quotes around environment variable values when generating environment_hash #156

Closed
wants to merge 1 commit into from

Conversation

keithpitt
Copy link

With something like this:

with upload_url: "http://foo.com?bar=foo&bar=foo" do
  ...
end

The environment_string generates a string that looks like this: UPLOAD_URL=http://foo.com?bar=foo&bar=foo which is invalid because it contains the ampersand (&). The solution I found was to wrap the values in quotes.

Was this an acceptable solution?

@leehambley
Copy link
Member

Is this fixed by #250? Can you check against the master branch please?

@leehambley leehambley closed this Feb 21, 2016
@leehambley
Copy link
Member

No feedback from @keithpitt for 10 months

@keithpitt
Copy link
Author

Ah sorry! I wasn't subscribed to the issue and I didn't get an @keithpitt notification. I didn't actually know this was fixed. Thanks!

@leehambley
Copy link
Member

No worries @keithpitt thanks for coming back to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants