summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-03-19 22:58:27 +0200
committerPaul Buetow <paul@buetow.org>2026-03-19 22:58:27 +0200
commitfb0791d88f32f25f23021493a26067c9aff22053 (patch)
treed6c48e88eae15d335e53045aae4933c12bb1243f
parentaa7f4a97b6a02484eacbdf9047bd4c570784df7a (diff)
task 256: support passphrase-protected key loading
-rw-r--r--internal/ssh/ssh.go16
-rw-r--r--internal/ssh/ssh_test.go113
2 files changed, 128 insertions, 1 deletions
diff --git a/internal/ssh/ssh.go b/internal/ssh/ssh.go
index a191fd5..25b9d28 100644
--- a/internal/ssh/ssh.go
+++ b/internal/ssh/ssh.go
@@ -6,6 +6,7 @@ import (
"crypto/rsa"
"crypto/x509"
"encoding/pem"
+ "errors"
"fmt"
"net"
"os"
@@ -121,7 +122,20 @@ func PrivateKeySigner(keyFile string) (gossh.Signer, error) {
}
key, err := gossh.ParsePrivateKey(buffer)
if err != nil {
- return nil, err
+ var passphraseMissingErr *gossh.PassphraseMissingError
+ if !errors.As(err, &passphraseMissingErr) {
+ return nil, err
+ }
+
+ passphrase := os.Getenv("DTAIL_KEY_PASSPHRASE")
+ if passphrase == "" {
+ return nil, err
+ }
+
+ key, err = gossh.ParsePrivateKeyWithPassphrase(buffer, []byte(passphrase))
+ if err != nil {
+ return nil, err
+ }
}
return key, nil
}
diff --git a/internal/ssh/ssh_test.go b/internal/ssh/ssh_test.go
new file mode 100644
index 0000000..f413145
--- /dev/null
+++ b/internal/ssh/ssh_test.go
@@ -0,0 +1,113 @@
+package ssh
+
+import (
+ "crypto/rand"
+ "crypto/rsa"
+ "encoding/pem"
+ "errors"
+ "os"
+ "path/filepath"
+ "testing"
+
+ gossh "golang.org/x/crypto/ssh"
+)
+
+func TestPrivateKeySignerLoadsUnencryptedKey(t *testing.T) {
+ keyFile := filepath.Join(t.TempDir(), "id_rsa")
+ privateKey, err := rsa.GenerateKey(rand.Reader, 1024)
+ if err != nil {
+ t.Fatalf("GenerateKey failed: %v", err)
+ }
+
+ if err := os.WriteFile(keyFile, EncodePrivateKeyToPEM(privateKey), 0o600); err != nil {
+ t.Fatalf("WriteFile failed: %v", err)
+ }
+
+ signer, err := PrivateKeySigner(keyFile)
+ if err != nil {
+ t.Fatalf("PrivateKeySigner failed: %v", err)
+ }
+ if signer == nil {
+ t.Fatalf("PrivateKeySigner returned nil signer")
+ }
+}
+
+func TestPrivateKeySignerLoadsEncryptedKeyWithEnvPassphrase(t *testing.T) {
+ keyFile := filepath.Join(t.TempDir(), "id_rsa")
+ privateKey, err := rsa.GenerateKey(rand.Reader, 1024)
+ if err != nil {
+ t.Fatalf("GenerateKey failed: %v", err)
+ }
+
+ block, err := gossh.MarshalPrivateKeyWithPassphrase(privateKey, "", []byte("secret-passphrase"))
+ if err != nil {
+ t.Fatalf("MarshalPrivateKeyWithPassphrase failed: %v", err)
+ }
+ if err := os.WriteFile(keyFile, pem.EncodeToMemory(block), 0o600); err != nil {
+ t.Fatalf("WriteFile failed: %v", err)
+ }
+
+ t.Setenv("DTAIL_KEY_PASSPHRASE", "secret-passphrase")
+
+ signer, err := PrivateKeySigner(keyFile)
+ if err != nil {
+ t.Fatalf("PrivateKeySigner failed: %v", err)
+ }
+ if signer == nil {
+ t.Fatalf("PrivateKeySigner returned nil signer")
+ }
+}
+
+func TestPrivateKeySignerReturnsPassphraseMissingWithoutEnv(t *testing.T) {
+ keyFile := filepath.Join(t.TempDir(), "id_rsa")
+ privateKey, err := rsa.GenerateKey(rand.Reader, 1024)
+ if err != nil {
+ t.Fatalf("GenerateKey failed: %v", err)
+ }
+
+ block, err := gossh.MarshalPrivateKeyWithPassphrase(privateKey, "", []byte("secret-passphrase"))
+ if err != nil {
+ t.Fatalf("MarshalPrivateKeyWithPassphrase failed: %v", err)
+ }
+ if err := os.WriteFile(keyFile, pem.EncodeToMemory(block), 0o600); err != nil {
+ t.Fatalf("WriteFile failed: %v", err)
+ }
+
+ _, err = PrivateKeySigner(keyFile)
+ if err == nil {
+ t.Fatalf("PrivateKeySigner succeeded without passphrase env")
+ }
+
+ var passphraseMissingErr *gossh.PassphraseMissingError
+ if !errors.As(err, &passphraseMissingErr) {
+ t.Fatalf("PrivateKeySigner returned %T, want PassphraseMissingError", err)
+ }
+}
+
+func TestPrivateKeySignerRejectsIncorrectEnvPassphrase(t *testing.T) {
+ keyFile := filepath.Join(t.TempDir(), "id_rsa")
+ privateKey, err := rsa.GenerateKey(rand.Reader, 1024)
+ if err != nil {
+ t.Fatalf("GenerateKey failed: %v", err)
+ }
+
+ block, err := gossh.MarshalPrivateKeyWithPassphrase(privateKey, "", []byte("secret-passphrase"))
+ if err != nil {
+ t.Fatalf("MarshalPrivateKeyWithPassphrase failed: %v", err)
+ }
+ if err := os.WriteFile(keyFile, pem.EncodeToMemory(block), 0o600); err != nil {
+ t.Fatalf("WriteFile failed: %v", err)
+ }
+
+ t.Setenv("DTAIL_KEY_PASSPHRASE", "wrong-passphrase")
+
+ _, err = PrivateKeySigner(keyFile)
+ if err == nil {
+ t.Fatalf("PrivateKeySigner succeeded with wrong passphrase env")
+ }
+
+ var passphraseMissingErr *gossh.PassphraseMissingError
+ if errors.As(err, &passphraseMissingErr) {
+ t.Fatalf("PrivateKeySigner returned PassphraseMissingError instead of parse failure")
+ }
+}