notkshitij/docker-credential-helpers
1
0
Fork 0
mirror of https://github.com/docker/docker-credential-helpers.git synced 2026-06-13 16:01:28 +05:30
Code Releases 1 Activity

Merge pull request #278 from thaJeztah/osxkeychain_typed_error

Browse Source 
osxkeychain: Delete(): return typed errors
This commit is contained in:
Sebastiaan van Stijn
2023-05-27 17:13:23 +02:00
committed by GitHub
parent 9ff5b6126b b21b69c8ee
commit f09e79d741
2 changed files with 27 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
osxkeychain/osxkeychain_darwin.go
+20 -16
View File
@@ -34,7 +34,7 @@ type Osxkeychain struct{}
// Add adds new credentials to the keychain. // Add adds new credentials to the keychain.
func (h Osxkeychain) Add(creds *credentials.Credentials) error { func (h Osxkeychain) Add(creds *credentials.Credentials) error {
h.Delete(creds.ServerURL) _ = h.Delete(creds.ServerURL) // ignore errors as existing credential may not exist.
s, err := splitServer(creds.ServerURL) s, err := splitServer(creds.ServerURL)
if err != nil { if err != nil {
@@ -66,10 +66,16 @@ func (h Osxkeychain) Delete(serverURL string) error {
} }
defer freeServer(s) defer freeServer(s)
errMsg := C.keychain_delete(s) if errMsg := C.keychain_delete(s); errMsg != nil {
if errMsg != nil {
defer C.free(unsafe.Pointer(errMsg)) defer C.free(unsafe.Pointer(errMsg))
return errors.New(C.GoString(errMsg)) switch goMsg := C.GoString(errMsg); goMsg {
case errCredentialsNotFound:
return credentials.NewErrCredentialsNotFound()
case errInteractionNotAllowed:
return ErrInteractionNotAllowed
default:
return errors.New(goMsg)
}
} }
return nil return nil
@@ -93,15 +99,14 @@ func (h Osxkeychain) Get(serverURL string) (string, string, error) {
errMsg := C.keychain_get(s, &usernameLen, &username, &secretLen, &secret) errMsg := C.keychain_get(s, &usernameLen, &username, &secretLen, &secret)
if errMsg != nil { if errMsg != nil {
defer C.free(unsafe.Pointer(errMsg)) defer C.free(unsafe.Pointer(errMsg))
goMsg := C.GoString(errMsg) switch goMsg := C.GoString(errMsg); goMsg {
if goMsg == errCredentialsNotFound { case errCredentialsNotFound:
return "", "", credentials.NewErrCredentialsNotFound() return "", "", credentials.NewErrCredentialsNotFound()
} case errInteractionNotAllowed:
if goMsg == errInteractionNotAllowed {
return "", "", ErrInteractionNotAllowed return "", "", ErrInteractionNotAllowed
default:
return "", "", errors.New(goMsg)
} }
return "", "", errors.New(goMsg)
} }
user := C.GoStringN(username, C.int(usernameLen)) user := C.GoStringN(username, C.int(usernameLen))
@@ -124,15 +129,14 @@ func (h Osxkeychain) List() (map[string]string, error) {
defer C.freeListData(&acctsC, listLenC) defer C.freeListData(&acctsC, listLenC)
if errMsg != nil { if errMsg != nil {
defer C.free(unsafe.Pointer(errMsg)) defer C.free(unsafe.Pointer(errMsg))
goMsg := C.GoString(errMsg) switch goMsg := C.GoString(errMsg); goMsg {
if goMsg == errCredentialsNotFound { case errCredentialsNotFound:
return make(map[string]string), nil return make(map[string]string), nil
} case errInteractionNotAllowed:
if goMsg == errInteractionNotAllowed {
return nil, ErrInteractionNotAllowed return nil, ErrInteractionNotAllowed
default:
return nil, errors.New(goMsg)
} }
return nil, errors.New(goMsg)
} }
var listLen int var listLen int
osxkeychain/osxkeychain_darwin_test.go
+7 -2
View File
@@ -205,9 +205,14 @@ func TestOSXKeychainHelperStoreRetrieve(t *testing.T) {
} }
func TestMissingCredentials(t *testing.T) { func TestMissingCredentials(t *testing.T) {
const nonExistingCred = "https://adsfasdf.invalid/asdfsdddd"
helper := Osxkeychain{} helper := Osxkeychain{}
_, _, err := helper.Get("https://adsfasdf.wrewerwer.com/asdfsdddd") _, _, err := helper.Get(nonExistingCred)
if !credentials.IsErrCredentialsNotFound(err) { if !credentials.IsErrCredentialsNotFound(err) {
t.Fatalf("expected ErrCredentialsNotFound, got %v", err) t.Errorf("expected ErrCredentialsNotFound, got %v", err)
}
err = helper.Delete(nonExistingCred)
if !credentials.IsErrCredentialsNotFound(err) {
t.Errorf("expected ErrCredentialsNotFound, got %v", err)
} }
} }