Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@shyim
Copy link
Contributor

@shyim shyim commented Dec 9, 2022

Adds support for hcloud from Hetzner well known hosting provider in Germany also expanding to US right now

Copy link
Contributor

@florisvdg florisvdg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! And thanks for adding a test as well.

DCKcode
DCKcode previously approved these changes Dec 9, 2022
@shyim
Copy link
Contributor Author

shyim commented Dec 17, 2022

Is fixed. Sorry for the delay :)

@SimonBarendse SimonBarendse merged commit ecf9bd0 into 1Password:main Dec 19, 2022
@notDavid
Copy link

Hi there, noob question;

when following the steps here i run:

$ hcloud context create my-project
The HCLOUD_TOKEN environment variable is set. Do you want to use the token from HCLOUD_TOKEN for the new context? (Y/n):
$ y

and then my API key gets permanently stored in plain text in /Users/david/.config/hcloud/cli.tom

Is this expected / intended behavior ?

@shyim
Copy link
Contributor Author

shyim commented Sep 14, 2025

yes. you need to import the key into 1password vault and it overrides it then. op plugin init hcloud

@notDavid
Copy link

notDavid commented Sep 15, 2025

@shyim Yep, i had done that, as i followed the steps in the guide i had linked to:

rm /Users/david/.config/hcloud/cli.tom

Guide:

op signin
op plugin init hcloud
# search in Password > "Prompt me for each new terminal session"
source ~/.config/op/plugins.sh
echo "source ~/.config/op/plugins.sh" >> ~/.bashrc && source ~/.bashrc

(So, it still saves the password in plain text.)

@notDavid
Copy link

For the record, it's the last step that saves the password in plain text:

hcloud context create my-project
# i then get prompted for TouchID

@notDavid
Copy link

@shyim Should i create a new issue for this?

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.

5 participants