fix: small number parsing improvements

Author: GrePod

internal/xrp/client.go

@@ -122,7 +122,7 @@ func (c *xrpClient) GetLatestBlockInfo(ctx context.Context) (*indexer.BlockInfo,
 // GetBlockTimestamp returns the Unix close time of the ledger at blockNum.
 func (c *xrpClient) GetBlockTimestamp(ctx context.Context, blockNum uint64) (uint64, error) {
 	getBlockParams := ledgerParams{
-		LedgerIndex:  strconv.Itoa(int(blockNum)),
+		LedgerIndex:  strconv.FormatUint(blockNum, 10),
 		Transactions: false,
 		Expand:       false,
 		OwnerFunds:   false,
@@ -138,7 +138,7 @@ func (c *xrpClient) GetBlockTimestamp(ctx context.Context, blockNum uint64) (uin
 // GetBlockResult returns the block header and all transactions for the ledger at blockNum.
 func (c *xrpClient) GetBlockResult(ctx context.Context, blockNum uint64) (*indexer.BlockResult[*Block, Transaction, struct{}], error) {
 	getBlockParams := ledgerParams{
-		LedgerIndex:  strconv.Itoa(int(blockNum)),
+		LedgerIndex:  strconv.FormatUint(blockNum, 10),
 		Transactions: true,
 		Expand:       true,
 		OwnerFunds:   false,

internal/xrp/transaction.go

@@ -83,7 +83,8 @@ func (tx *transaction) isNativePayment() (bool, error) {
 		if err := json.Unmarshal(tx.Amount, &amountStr); err != nil {
 			return false, fmt.Errorf("unmarshal amount string: %w", err)
 		}
-		_, err := strconv.Atoi(amountStr)
+
+		_, err := strconv.ParseUint(amountStr, 10, 64)
 		return err == nil, nil
 	}
 
@@ -98,7 +99,7 @@ func (tx *transaction) isNativePayment() (bool, error) {
 // Destination address, or nil if no destination is set.
 func (tx *transaction) destinationAddressHash() *string {
 	if len(tx.Destination) > 0 {
-		hash := crypto.Keccak256Hash([]byte(tx.Destination)).Hex()[2:]
+		hash := strings.ToLower(crypto.Keccak256Hash([]byte(tx.Destination)).Hex()[2:])
 		return &hash
 	}