Skip to content

Commit

Permalink
fix(cmd/blob): fix namespace parsing in cli (#4003)
Browse files Browse the repository at this point in the history
  • Loading branch information
vgonkivs authored Dec 12, 2024
1 parent 7af07bd commit 01bb3e2
Showing 1 changed file with 16 additions and 10 deletions.
26 changes: 16 additions & 10 deletions nodebuilder/blob/cmd/blob.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,11 +43,11 @@ var getCmd = &cobra.Command{
Short: "Returns the blob for the given namespace by commitment at a particular height.\n" +
"Note:\n* Both namespace and commitment input parameters are expected to be in their hex representation.",
PreRunE: func(_ *cobra.Command, args []string) error {
if !strings.HasPrefix(args[1], "0x") {
return fmt.Errorf("only hex namespace is supported")
if !strings.HasPrefix(args[0], "0x") {
args[0] = "0x" + args[0]
}
if !strings.HasPrefix(args[2], "0x") {
return fmt.Errorf("only hex commitment is supported")
if !strings.HasPrefix(args[1], "0x") {
args[1] = "0x" + args[1]
}
return nil
},
Expand Down Expand Up @@ -84,8 +84,11 @@ var getAllCmd = &cobra.Command{
Short: "Returns all blobs for the given namespace at a particular height.\n" +
"Note:\n* Namespace input parameter is expected to be in its hex representation.",
PreRunE: func(_ *cobra.Command, args []string) error {
if !strings.HasPrefix(args[0], "0x") {
args[0] = "0x" + args[0]
}
if !strings.HasPrefix(args[1], "0x") {
return fmt.Errorf("only hex namespace is supported")
args[1] = "0x" + args[1]
}
return nil
},
Expand Down Expand Up @@ -136,7 +139,10 @@ var submitCmd = &cobra.Command{
},
PreRunE: func(_ *cobra.Command, args []string) error {
if !strings.HasPrefix(args[0], "0x") {
return fmt.Errorf("only hex namespace is supported")
args[0] = "0x" + args[0]
}
if !strings.HasPrefix(args[1], "0x") {
args[1] = "0x" + args[1]
}
return nil
},
Expand Down Expand Up @@ -235,11 +241,11 @@ var getProofCmd = &cobra.Command{
Short: "Retrieves the blob in the given namespaces at the given height by commitment and returns its Proof.\n" +
"Note:\n* Both namespace and commitment input parameters are expected to be in their hex representation.",
PreRunE: func(_ *cobra.Command, args []string) error {
if !strings.HasPrefix(args[1], "0x") {
return fmt.Errorf("only hex namespace is supported")
if !strings.HasPrefix(args[0], "0x") {
args[0] = "0x" + args[0]
}
if !strings.HasPrefix(args[2], "0x") {
return fmt.Errorf("only hex commitment is supported")
if !strings.HasPrefix(args[1], "0x") {
args[1] = "0x" + args[1]
}
return nil
},
Expand Down

0 comments on commit 01bb3e2

Please sign in to comment.