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

README Update for encrypted SSH key usage #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nandorholozsnyak
Copy link

init.sh contains this:

if [ -n "$IDENC" ]
then
  ID=$(echo $IDENC |base64 -d)
  IDPUB=$(echo $IDPUBENC |base64 -d)
fi

And the README.md said that to use base64 encoded values in ID and IDPUB environment variables and that is wrong! These fields need pure raw ssh keys, if you want to use base64 encoded values please use the IDENC and IDPUBENC for safety.

@nandorholozsnyak
Copy link
Author

It's a really good and handy tool, please keep it up-to-date :D

@gjonespf
Copy link
Owner

Hey @Holi60k - cheers for this. Yeah, just a fork of the original tool with merges from good forks, as original hasn't been updated in ages. I don't actually use these SSH parameters atm, I recall tinkering with trying to get this working consistently and having issues. Will test these out and merge if all good.

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.

2 participants