Merge pull request Stride-Labs#101 from Stride-Labs/certik-misc

Misc certik changes (from prev weeks)

x/interchainquery/keeper/keeper.go

@@ -135,6 +135,12 @@ func (k *Keeper) MakeRequest(ctx sdk.Context, connection_id string, chain_id str
 	// ======================================================================================================================
 	// Perform basic validation on the query input
 
+	// today we only support queries at the latest block height on the host zone, specified by "height=0"
+
+	if height != 0 {
+		return fmt.Errorf("ICQ query height must be 0! Found a query at non-zero height %d", height)
+	}
+
 	// connection id cannot be empty and must begin with "connection"
 	if connection_id == "" {
 		k.Logger(ctx).Error("[ICQ Validation Check] Failed! connection id cannot be empty")

x/stakeibc/keeper/msg_server_submit_tx.go

@@ -166,8 +166,8 @@ func (k Keeper) UpdateWithdrawalBalance(ctx sdk.Context, zoneInfo types.HostZone
 		sdk.NewInt(-1),
 		types.ModuleName,
 		"withdrawalbalance",
-		0, //ttl
-		0, //height
+		0, // ttl
+		0, // height always 0 (which means current height)
 	)
 	if err != nil {
 		k.Logger(ctx).Error("Error querying for withdrawal balance", "error", err)