-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Lando creates empty .ssh folder in app root #134
Comments
A PR would be great @michaellopez, thanks for pointing it out! |
@reynoldsalec Here you go #135 |
You can avoid creating rogue .ssh paths by using an absolute path as the value... eg home: '/Users/al/.lando-home-jail' then it'll go and create the .ssh dir in that root. the .ssh dir is used to load your id_rsa up... so, if you change the home dir, you should probably also copy \cp -fR ~/.gitconfig ~/.lando-home-jail |
@sonuku That's pretty clever, thanks |
Thanks for Lando! I would like to report a simple but annoying bug.
Due to 582e22a when setting
home: ''
in~/.lando/config.yml
to avoid mounting the home folder as described in lando/lando#2635 (comment) Lando creates an empty folder named.ssh
in the app root (next to.lando.yml
)This happens since
lando.config.home
is an empty string, sopath.join(lando.config.home, '.ssh')
evaluates to.ssh
and therefore creates the folder in the working directory.Would you like a PR or is this simple enough for you to quick fix with something like for lines 64-66:
Thanks for all your efforts!
The text was updated successfully, but these errors were encountered: