use correct type; just look for len() instead of nil check (#2576)

bobcallaway · web-flow · commit d2372a3781b5 · 2025-08-11T22:25:12.000Z
Signed-off-by: Bob Callaway &lt;bcallaway@google.com&gt;
diff --git a/pkg/api/api.go b/pkg/api/api.go
@@ -148,7 +148,7 @@ func NewAPI(treeID uint) (*API, error) {
 
 	algorithmsOption := viper.GetStringSlice("client-signing-algorithms")
 	var algorithms []v1.PublicKeyDetails
-	if algorithmsOption == nil {
+	if len(algorithmsOption) == 0 {
 		algorithms = DefaultClientSigningAlgorithms
 	} else {
 		for _, a := range algorithmsOption {
diff --git a/pkg/api/entries.go b/pkg/api/entries.go
@@ -314,7 +314,7 @@ func createLogEntry(params entries.CreateLogEntryParams) (models.LogEntry, middl
 			}
 		case *rsa.PublicKey:
 			publicKeyInfo = fmt.Sprintf("rsa public key: %T/%v", pKey, pKey.Size())
-		case *ed25519.PublicKey:
+		case ed25519.PublicKey:
 			publicKeyInfo = fmt.Sprintf("ed25519 public key: %T", pKey)
 		default:
 			publicKeyInfo = fmt.Sprintf("public key: %T", pKey)
