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.

Sign up for GitHub

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

Jump to bottom

Empty strings parsed as \"\" #612

Closed
dackmin opened this issue Feb 2, 2022 · 3 comments
Closed

Empty strings parsed as \"\" #612

dackmin opened this issue Feb 2, 2022 · 3 comments

Comments

@dackmin
Copy link

dackmin commented Feb 2, 2022

Hi,

Starting with v15 empty strings seem to be wrongly parsed:
https://runkit.com/dackmin/dotenv-issue

v14 -> ''
v15 -> '""'

Is this normal and not reflected in the changelog? or not?

Thanks
@motdotla
Copy link
Owner

motdotla commented Feb 2, 2022 •
edited
Loading

All these many years and we never had a test for that scenario. Apologies for this regression, and thank you for catching!

PR is here and will go out as v15.0.1 shortly. #614

@dackmin
Copy link
Author

dackmin commented Feb 2, 2022

I had the unfortunate chance to witness it in production lmao. But at least that helped catch it and I added a test on my side too 😄
thanks for the fix!

@motdotla
Copy link
Owner

motdotla commented Feb 2, 2022

Ah, so sorry! It's released: https://www.npmjs.com/package/dotenv/v/15.0.1 v15.0.1

@motdotla motdotla closed this as completed Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@motdotla @dackmin