diff --git a/internal/secret/keychainunlocker_stub.go b/internal/secret/keychainunlocker_stub.go index e6c6fb7..6e79370 100644 --- a/internal/secret/keychainunlocker_stub.go +++ b/internal/secret/keychainunlocker_stub.go @@ -4,6 +4,8 @@ package secret import ( + "fmt" + "filippo.io/age" "github.com/awnumar/memguard" "github.com/spf13/afero" @@ -22,52 +24,59 @@ type KeychainUnlocker struct { fs afero.Fs } -// GetIdentity panics on non-Darwin platforms +var errKeychainNotSupported = fmt.Errorf("keychain unlockers are only supported on macOS") + +// GetIdentity returns an error on non-Darwin platforms func (k *KeychainUnlocker) GetIdentity() (*age.X25519Identity, error) { - panic("keychain unlockers are only supported on macOS") + return nil, errKeychainNotSupported } -// GetType panics on non-Darwin platforms +// GetType returns the unlocker type func (k *KeychainUnlocker) GetType() string { - panic("keychain unlockers are only supported on macOS") + return "keychain" } -// GetMetadata panics on non-Darwin platforms +// GetMetadata returns the unlocker metadata func (k *KeychainUnlocker) GetMetadata() UnlockerMetadata { - panic("keychain unlockers are only supported on macOS") + return k.Metadata } -// GetDirectory panics on non-Darwin platforms +// GetDirectory returns the unlocker directory func (k *KeychainUnlocker) GetDirectory() string { - panic("keychain unlockers are only supported on macOS") + return k.Directory } // GetID returns the unlocker ID func (k *KeychainUnlocker) GetID() string { - panic("keychain unlockers are only supported on macOS") + return fmt.Sprintf("%s-keychain", k.Metadata.CreatedAt.Format("2006-01-02.15.04")) } -// GetKeychainItemName panics on non-Darwin platforms +// GetKeychainItemName returns an error on non-Darwin platforms func (k *KeychainUnlocker) GetKeychainItemName() (string, error) { - panic("keychain unlockers are only supported on macOS") + return "", errKeychainNotSupported } -// Remove panics on non-Darwin platforms +// Remove returns an error on non-Darwin platforms func (k *KeychainUnlocker) Remove() error { - panic("keychain unlockers are only supported on macOS") + return errKeychainNotSupported } -// NewKeychainUnlocker panics on non-Darwin platforms +// NewKeychainUnlocker creates a stub KeychainUnlocker on non-Darwin platforms. +// The returned instance's methods that require macOS functionality will return errors. func NewKeychainUnlocker(fs afero.Fs, directory string, metadata UnlockerMetadata) *KeychainUnlocker { - panic("keychain unlockers are only supported on macOS") + return &KeychainUnlocker{ + Directory: directory, + Metadata: metadata, + fs: fs, + } } -// CreateKeychainUnlocker panics on non-Darwin platforms -func CreateKeychainUnlocker(fs afero.Fs, stateDir string) (*KeychainUnlocker, error) { - panic("keychain unlockers are only supported on macOS") +// CreateKeychainUnlocker returns an error on non-Darwin platforms +func CreateKeychainUnlocker(_ afero.Fs, _ string) (*KeychainUnlocker, error) { + return nil, errKeychainNotSupported } -// getLongTermPrivateKey panics on non-Darwin platforms -func getLongTermPrivateKey(fs afero.Fs, vault VaultInterface) (*memguard.LockedBuffer, error) { - panic("keychain unlockers are only supported on macOS") +// getLongTermPrivateKey returns an error on non-Darwin platforms +func getLongTermPrivateKey(_ afero.Fs, _ VaultInterface) (*memguard.LockedBuffer, error) { + return nil, errKeychainNotSupported }