Merge pull request #437 from LN-Zap/fix/case-insensitive-asset-parsing-clean

fix: peersync - case insensitive asset parsing

Author: YusukeShimizu

peersync/peer.go

@@ -4,6 +4,7 @@ package peersync
 import (
 	"errors"
 	"fmt"
+	"strings"
 	"time"
 
 	"github.com/elementsproject/peerswap/premium"
@@ -34,7 +35,8 @@ var (
 
 // NewAsset converts a ticker into the corresponding enum value.
 func NewAsset(value string) (Asset, error) {
-	if asset, ok := stringToAsset[value]; ok {
+	normalized := strings.ToUpper(strings.TrimSpace(value))
+	if asset, ok := stringToAsset[normalized]; ok {
 		return asset, nil
 	}
 	return AssetUnknown, fmt.Errorf("unsupported asset: %s", value)

peersync/peer_test.go

@@ -275,8 +275,21 @@ func TestAssetValidation(t *testing.T) {
 		t.Fatalf("expected asset LBTC string to be LBTC, got %s", b.String())
 	}
 
-	if _, err = NewAsset("lower"); err == nil {
-		t.Fatalf("expected error for lowercase asset")
+	// v5.0.0 peers send lowercase symbols — must be accepted
+	c, err := NewAsset("btc")
+	if err != nil {
+		t.Fatalf("expected no error for lowercase btc, got %v", err)
+	}
+	if c != AssetBTC {
+		t.Fatalf("expected lowercase btc to resolve to AssetBTC")
+	}
+
+	d, err := NewAsset("lbtc")
+	if err != nil {
+		t.Fatalf("expected no error for lowercase lbtc, got %v", err)
+	}
+	if d != AssetLBTC {
+		t.Fatalf("expected lowercase lbtc to resolve to AssetLBTC")
 	}
 
 	if _, err = NewAsset("TOO-LONG-ASSET"); err == nil {