Merge pull request #31 from cedrickah/feature/block-get-and-head-body

whatwewant · web-flow · commit ecf7555875c5 · 2025-06-23T11:38:46.000+08:00
feat: block body when request method is GET or HEAD
diff --git a/get.go b/get.go
@@ -9,5 +9,9 @@ func Get(url string, config ...interface{}) (*Response, error) {
 		return nil, ErrTooManyArguments
 	}
 
+	if c.Body != nil {
+        panic("Request with GET method cannot have body")
+    }
+
 	return New().Get(url, c).Execute()
 }
diff --git a/head.go b/head.go
@@ -9,5 +9,9 @@ func Head(url string, config ...interface{}) (*Response, error) {
 		return nil, ErrTooManyArguments
 	}
 
+	if c.Body != nil {
+        panic("Request with HEAD method cannot have body")
+    }
+
 	return New().Head(url, c).Execute()
 }

Original file line number	Diff line number	Diff line change
`@@ -9,5 +9,9 @@ func Get(url string, config ...interface{}) (*Response, error) {`
`9`	`9`	`return nil, ErrTooManyArguments`
`10`	`10`	`}`
`11`	`11`
	`12`	`+ if c.Body != nil {`
	`13`	`+ panic("Request with GET method cannot have body")`
	`14`	`+ }`
	`15`	`+`
`12`	`16`	`return New().Get(url, c).Execute()`
`13`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,5 +9,9 @@ func Head(url string, config ...interface{}) (*Response, error) {`
`9`	`9`	`return nil, ErrTooManyArguments`
`10`	`10`	`}`
`11`	`11`
	`12`	`+ if c.Body != nil {`
	`13`	`+ panic("Request with HEAD method cannot have body")`
	`14`	`+ }`
	`15`	`+`
`12`	`16`	`return New().Head(url, c).Execute()`
`13`	`17`	`}`