address some comments

Layr-Labs · Apr 26, 2024 · cca159b · cca159b
1 parent 602fe25
commit cca159b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/node/plugin/cmd/main.go b/node/plugin/cmd/main.go
@@ -132,10 +132,10 @@ func pluginOps(ctx *cli.Context) {
 		}
 		quorumIds, err := tx.GetRegisteredQuorumIdsForOperator(context.Background(), operatorID)
 		if err != nil {
-			log.Printf("Error: failed to get quorum(s) for operatorID: %x, operator address: %x, error: %v", operatorID, sk.Address, err)
+			log.Printf("Error: failed to get quorum(s) for operatorID: %x, operator address: %x, error: %v", operatorID, operatorAddress.Hex(), err)
 			return
 		}
-		log.Printf("Info: operator ID: %x, operator address: %x, current quorums: %v", operatorID, operatorAddress, quorumIds)
+		log.Printf("Info: operator ID: %x, operator address: %x, current quorums: %v", operatorID, operatorAddress.Hex(), quorumIds)
 		return
 	}
 

diff --git a/node/plugin/config.go b/node/plugin/config.go
@@ -150,7 +150,10 @@ func NewConfig(ctx *cli.Context) (*Config, error) {
 		return nil, errors.New("unsupported operation type")
 	}
 
-	if op != OperationListQuorums && len(ctx.GlobalString(EcdsaKeyFileFlag.Name)) == 0 && len(ctx.GlobalString(EcdsaKeyPasswordFlag.Name)) == 0 {
+	// ECDSA key is only required for opt-in, opt-out and update-socket operations
+	if (op == OperationOptIn || op == OperationOptOut || op == OperationUpdateSocket) &&
+		len(ctx.GlobalString(EcdsaKeyFileFlag.Name)) == 0 &&
+		len(ctx.GlobalString(EcdsaKeyPasswordFlag.Name)) == 0 {
 		return nil, errors.New("opt-in, opt-out and update-socket operations require ECDSA key file and password")
 	}