-
Notifications
You must be signed in to change notification settings - Fork 12
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
"env:set" does not support non-alphabetical characters #33
Comments
Ah, well spotted! Seems like you've already identified the bug, can you submit a PR to fix this? If not, I'll have a look at this later next week 🙂 |
Sorry @svenluijten, a little busy at this time for PR. |
No worries @svytas! I'll pick it up as soon as I can make some time 😄 |
fixed by #38 |
Hi, still it works strange - looks like I cannot change value if there are no numbers in it: vagrant@homestead:/test$ php artisan env:set MAIL_PASSWORD 2 |
That's... very strange. This package desperately needs a rewrite / big refactor anyway, I'll see what I can do! |
Did you try to set a value with a plus character? The result will surprise you:
.env file:
Executing command:
php artisan env:set MAIL_PASSWORD ABCDEF+
Result file looks like this:
If your old value has other charaters, you get error:
.env file:
Executing command:
php artisan env:set MAIL_PASSWORD ABCDEF+
got error
Could not set the value in your .env file, reverting...
The problem is
preg_replace
function...The text was updated successfully, but these errors were encountered: