Skip to content

Support SSH keys during dedicated server installs - #2

Merged
sjmiller609 merged 2 commits into
mainfrom
hypeship/add-install-ssh-keys
Aug 26, 2026
Merged

sjmiller609 merged 2 commits into
mainfrom
hypeship/add-install-ssh-keys

Conversation

@sjmiller609

@sjmiller609 sjmiller609 commented Aug 26, 2026 •

Copy link
Copy Markdown
Contributor

summary

  • add a repeatable --ssh-key option to dedicated-servers install
  • send provided keys through Leaseweb’s newline-delimited sshKeys install field
  • cover single-key and multi-key requests

validation

  • go test ./...
  • go vet ./...
  • verified generated command help

Note

Low Risk
Small, optional CLI extension to an existing install command with tests; no auth or broad behavioral changes beyond the new payload field.

Overview
Adds repeatable --ssh-key flags to dedicated-servers install so public keys can be passed at OS install time.

When any keys are provided, the install handler includes them in the API payload as sshKeys, with multiple values joined by newlines to match Leaseweb’s format. DisableSliceFlagSeparator is enabled on the command so slice parsing does not split key material on commas (e.g. in key comments).

Integration tests assert the install POST body for a single key and for multiple --ssh-key repetitions.

Reviewed by Cursor Bugbot for commit 132eeb2. Bugbot is set up for automated code reviews on this repo. Configure here.

@sjmiller609
sjmiller609 marked this pull request as ready for review August 26, 2026 13:41

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 322c9aa. Configure here.

Comment thread pkg/cmd/dedicated_servers.go
@sjmiller609
sjmiller609 merged commit f9d8408 into main Aug 26, 2026
3 checks passed
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.

1 participant