fix: dsn without a HTTP port

uptrace/dsn.go

@@ -24,14 +24,14 @@ func (dsn *DSN) SiteURL() string {
 	if dsn.Host == "uptrace.dev" {
 		return "https://app.uptrace.dev"
 	}
-	return dsn.Scheme + "://" + net.JoinHostPort(dsn.Host, dsn.HTTPPort)
+	return dsn.Scheme + "://" + joinHostPort(dsn.Host, dsn.HTTPPort)
 }
 
 func (dsn *DSN) OTLPEndpoint() string {
 	if dsn.Host == "uptrace.dev" {
 		return "otlp.uptrace.dev:4317"
 	}
-	return net.JoinHostPort(dsn.Host, dsn.GRPCPort)
+	return joinHostPort(dsn.Host, dsn.GRPCPort)
 }
 
 func ParseDSN(dsnStr string) (*DSN, error) {
@@ -84,3 +84,10 @@ func ParseDSN(dsnStr string) (*DSN, error) {
 
 	return &dsn, nil
 }
+
+func joinHostPort(host, port string) string {
+	if port == "" {
+		return host
+	}
+	return net.JoinHostPort(host, port)
+}

uptrace/dsn_test.go

@@ -17,9 +17,14 @@ func TestParseDSN(t *testing.T) {
 	}
 
 	tests := []Test{
-		{"https://[email protected]/1", "otlp.uptrace.dev:4317", "https://app.uptrace.dev"},
-		{"https://[email protected]/1", "otlp.uptrace.dev:4317", "https://app.uptrace.dev"},
-		{"https://key@localhost:1234/1", "localhost:1234", "https://localhost:1234"},
+		{"https://[email protected]/1", "otlp.uptrace.dev:4317", "https://app.uptrace.dev"},
+		{"https://[email protected]/1", "otlp.uptrace.dev:4317", "https://app.uptrace.dev"},
+		{
+			"https://[email protected]/1?grpc=4317",
+			"demo.uptrace.dev:4317",
+			"https://demo.uptrace.dev",
+		},
+		{"https://token@localhost:1234/1", "localhost:1234", "https://localhost:1234"},
 		{"http://token@localhost:14317/project_id", "localhost:14317", "http://localhost:14318"},
 		{
 			"https://[email protected]:4317/1",