Default value of JSON RPC flag in commands (0xPolygon#1939)

* Fixed default value of JSON RPC flag in commands

* Added JSON RPC flag value validation in bridge commands

command/bridge/common/params.go

@@ -86,6 +86,19 @@ func (p *BridgeParams) RegisterCommonFlags(cmd *cobra.Command) {
 	)
 }
 
+func (p *BridgeParams) Validate() error {
+	if p == nil {
+		return nil
+	}
+
+	_, err := cmdHelper.ParseJSONRPCAddress(p.JSONRPCAddr)
+	if err != nil {
+		return fmt.Errorf("failed to parse json rpc address. Error: %w", err)
+	}
+
+	return nil
+}
+
 type ERC20BridgeParams struct {
 	*BridgeParams
 	Amounts []string
@@ -96,6 +109,10 @@ func NewERC20BridgeParams() *ERC20BridgeParams {
 }
 
 func (bp *ERC20BridgeParams) Validate() error {
+	if err := bp.BridgeParams.Validate(); err != nil {
+		return err
+	}
+
 	if len(bp.Receivers) != len(bp.Amounts) {
 		return errInconsistentAmounts
 	}
@@ -113,6 +130,10 @@ func NewERC721BridgeParams() *ERC721BridgeParams {
 }
 
 func (bp *ERC721BridgeParams) Validate() error {
+	if err := bp.BridgeParams.Validate(); err != nil {
+		return err
+	}
+
 	if len(bp.Receivers) != len(bp.TokenIDs) {
 		return errInconsistentTokenIds
 	}
@@ -131,6 +152,10 @@ func NewERC1155BridgeParams() *ERC1155BridgeParams {
 }
 
 func (bp *ERC1155BridgeParams) Validate() error {
+	if err := bp.BridgeParams.Validate(); err != nil {
+		return err
+	}
+
 	if len(bp.Receivers) != len(bp.Amounts) {
 		return errInconsistentAmounts
 	}

command/helper/helper.go

@@ -209,7 +209,7 @@ func ParseGRPCAddress(grpcAddress string) (*net.TCPAddr, error) {
 func RegisterJSONRPCFlag(cmd *cobra.Command) {
 	cmd.PersistentFlags().String(
 		command.JSONRPCFlag,
-		fmt.Sprintf("%s:%d", AllInterfacesBinding, server.DefaultJSONRPCPort),
+		fmt.Sprintf("http://%s:%d", AllInterfacesBinding, server.DefaultJSONRPCPort),
 		"the JSON-RPC interface",
 	)
 }

command/rootchain/validators/params.go

@@ -18,6 +18,10 @@ type validatorInfoParams struct {
 }
 
 func (v *validatorInfoParams) validateFlags() error {
+	if _, err := helper.ParseJSONRPCAddress(v.jsonRPC); err != nil {
+		return fmt.Errorf("failed to parse json rpc address. Error: %w", err)
+	}
+
 	return sidechainHelper.ValidateSecretFlags(v.accountDir, v.accountConfig)
 }
 

command/rootchain/withdraw/params.go

@@ -29,6 +29,10 @@ func (v *withdrawParams) validateFlags() (err error) {
 		return err
 	}
 
+	if _, err = helper.ParseJSONRPCAddress(v.jsonRPC); err != nil {
+		return fmt.Errorf("failed to parse json rpc address. Error: %w", err)
+	}
+
 	return sidechainHelper.ValidateSecretFlags(v.accountDir, v.accountConfig)
 }
 

command/sidechain/rewards/params.go

@@ -20,6 +20,10 @@ type withdrawRewardResult struct {
 }
 
 func (w *withdrawRewardsParams) validateFlags() error {
+	if _, err := helper.ParseJSONRPCAddress(w.jsonRPC); err != nil {
+		return fmt.Errorf("failed to parse json rpc address. Error: %w", err)
+	}
+
 	return sidechainHelper.ValidateSecretFlags(w.accountDir, w.accountConfig)
 }
 

command/sidechain/unstaking/params.go

@@ -27,6 +27,10 @@ func (v *unstakeParams) validateFlags() (err error) {
 		return err
 	}
 
+	if _, err = helper.ParseJSONRPCAddress(v.jsonRPC); err != nil {
+		return fmt.Errorf("failed to parse json rpc address. Error: %w", err)
+	}
+
 	return sidechainHelper.ValidateSecretFlags(v.accountDir, v.accountConfig)
 }
 

command/sidechain/withdraw/params.go

@@ -16,6 +16,10 @@ type withdrawParams struct {
 }
 
 func (w *withdrawParams) validateFlags() error {
+	if _, err := helper.ParseJSONRPCAddress(w.jsonRPC); err != nil {
+		return fmt.Errorf("failed to parse json rpc address. Error: %w", err)
+	}
+
 	return sidechainHelper.ValidateSecretFlags(w.accountDir, w.accountConfig)
 }