Fix: vmess http method may empty

T-halo · Aug 3, 2023 · 47b6eb1 · 47b6eb1
1 parent 9e78137
commit 47b6eb1
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/common/util/manipulation.go b/common/util/manipulation.go
@@ -0,0 +1,8 @@
+package util
+
+import "github.com/samber/lo"
+
+func EmptyOr[T comparable](v T, def T) T {
+	ret, _ := lo.Coalesce(v, def)
+	return ret
+}
diff --git a/transport/vmess/http.go b/transport/vmess/http.go
@@ -8,6 +8,8 @@ import (
 	"net"
 	"net/http"
 	"net/textproto"
+
+	"github.com/Dreamacro/clash/common/util"
 )
 
 type httpConn struct {
@@ -58,7 +60,7 @@ func (hc *httpConn) Write(b []byte) (int, error) {
 	}
 
 	u := fmt.Sprintf("http://%s%s", host, path)
-	req, _ := http.NewRequest(hc.cfg.Method, u, bytes.NewBuffer(b))
+	req, _ := http.NewRequest(util.EmptyOr(hc.cfg.Method, http.MethodGet), u, bytes.NewBuffer(b))
 	for key, list := range hc.cfg.Headers {
 		req.Header.Set(key, list[rand.Intn(len(list))])
 	}