Implement missing functions in NotarySigner to fully satisfy CryptoService interface

These missing functions aren't used, so they're not strictly necessary
at this point. However, it's cleaner to fully implement the interface
than to have functions that don't work with TODO comments.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
This commit is contained in:
Aaron Lehmann 2015-07-20 09:57:44 -07:00
parent 9d31d343f3
commit a9f12f52f8
1 changed files with 7 additions and 5 deletions

View File

@ -1,7 +1,6 @@
package signer package signer
import ( import (
"errors"
"net" "net"
"github.com/Sirupsen/logrus" "github.com/Sirupsen/logrus"
@ -75,12 +74,15 @@ func (trust *NotarySigner) Create(role string, algorithm data.KeyAlgorithm) (dat
// RemoveKey deletes a key // RemoveKey deletes a key
func (trust *NotarySigner) RemoveKey(keyid string) error { func (trust *NotarySigner) RemoveKey(keyid string) error {
//TODO(aaronl): Not implemented yet _, err := trust.kmClient.DeleteKey(context.Background(), &pb.KeyID{ID: keyid})
return errors.New("DeleteKey not implemented in NotarySigner") return err
} }
// GetKey retrieves a key // GetKey retrieves a key
func (trust *NotarySigner) GetKey(keyid string) data.PublicKey { func (trust *NotarySigner) GetKey(keyid string) data.PublicKey {
//TODO(aaronl): Not implemented yet publicKey, err := trust.kmClient.GetKeyInfo(context.Background(), &pb.KeyID{ID: keyid})
return nil if err != nil {
return nil
}
return data.NewPublicKey(data.KeyAlgorithm(publicKey.KeyInfo.Algorithm.Algorithm), publicKey.PublicKey)
} }