nip11: url normalization improvement and Fetch test.

nip11/fetch.go

@@ -5,7 +5,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"net/url"
 	"strings"
 	"time"
 )
@@ -20,21 +19,12 @@ func Fetch(ctx context.Context, u string) (info *RelayInformationDocument, err e
 	}
 
 	// normalize URL to start with http:// or https://
-	if !strings.HasPrefix(u, "http") && !strings.HasPrefix(u, "ws") {
+	if strings.HasPrefix(u, "ws") {
-		u = "wss://" + u
+		u = "http" + u[2:]
 	}
-	p, err := url.Parse(u)
+	u = strings.TrimRight(u, "/")
-	if err != nil {
-		return nil, fmt.Errorf("cannot parse url: %s", u)
-	}
-	if p.Scheme == "ws" {
-		p.Scheme = "http"
-	} else if p.Scheme == "wss" {
-		p.Scheme = "https"
-	}
-	p.Path = strings.TrimRight(p.Path, "/")
 
-	req, err := http.NewRequestWithContext(ctx, http.MethodGet, p.String(), nil)
+	req, err := http.NewRequestWithContext(ctx, http.MethodGet, u, nil)
 
 	// add the NIP-11 header
 	req.Header.Add("Accept", "application/nostr+json")

nip11/nip11_test.go

@@ -1,6 +1,9 @@
 package nip11
 
-import "testing"
+import (
+	"context"
+	"testing"
+)
 
 func TestAddSupportedNIP(t *testing.T) {
 	info := RelayInformationDocument{}
@@ -28,3 +31,14 @@ func TestAddSupportedNIP(t *testing.T) {
 		}
 	}
 }
+
+func TestFetch(t *testing.T) {
+	res, err := Fetch(context.Background(), "wss://relay.nostr.bg")
+	if err != nil || res.Name == "" {
+		t.Errorf("failed to fetch from wss")
+	}
+	res, err = Fetch(context.Background(), "https://relay.nostr.bg")
+	if err != nil || res.Name == "" {
+		t.Errorf("failed to fetch from https")
+	}
+}