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

Fix to allow = as a character in the env file #41

Closed
wants to merge 4 commits into from

Conversation

philippsimon
Copy link

I had the problem that i was storing a part of a URL in my .env file and it was cut off. With this fix it'll work.

here also a fix for the test file envs-commented.test.js

var parsedHash = envs.KeyValue(
  '#commented heading. \n' +
  'key = "quoted#hash" \n' +
  'key2 = stripped#comment \n' +
  'key3 = =equal=in string= \n'
)

assert.equal(parsedHash['key'], 'quoted#hash')
assert.equal(parsedHash['key2'], 'stripped')
assert.equal(parsedHash['key3'], '=equal=in string=')

@rmg
Copy link
Member

rmg commented May 2, 2014

Oops, looks like I got master out of sync. This bug was actually fixed 3 weeks ago when I released 0.3.0 but I forgot to push the changes back to GitHub.

Can you confirm v0.3.0 from npm fixes this for you?

@rmg
Copy link
Member

rmg commented May 2, 2014

And now I've gone and force pushed 3 week old commits to master... I'm winning today!

Anyway, fix is in 4c1b7df.

@philippsimon
Copy link
Author

works - thanks!!

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.

None yet

2 participants