adonisjs / env

Framework agnostic environment variables parser and validator
https://docs.adonisjs.com/guides/environment-variables
MIT License
37 stars 10 forks source link

Unable to escape single sequences #25

Closed thetutlage closed 3 years ago

thetutlage commented 3 years ago

Discussed in https://github.com/adonisjs/core/discussions/2858

The single $ sequences are not escaped when using the escape slash '\'. For example:

PASSWORD=pa\$word

Return

pas$

Expected

pas$word
varun-nambiar commented 3 years ago

Hey, I have created a pull request here - https://github.com/adonisjs/env/pull/26


Is there a guide/documentation/read/reference on how to do patch fixes on any packages
For instance, when in Laravel or another frameworks, I can directly go into the vendor/ and make changes to the files until a bug fix is available
Something similar for node..

thetutlage commented 3 years ago

I can directly go into the vendor/ and make changes to the files until a bug fix is available

But I think your changes will be lost once you run composer install again?

thetutlage commented 3 years ago

Released as 3.0.4