daren.hsu/webhook
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Allow charset to be defined in Content-Type header

Browse Source 
The payload couldn't be parsed when charset was present in the
`Content-Type` header. The content type should begin with the MIME type
so we now check if the content type starts with `application/json` or
`application/x-www-form-urlencoded`.

This closes #20
This commit is contained in:
Kevin Le Brun 2015-03-29 19:58:38 +02:00
parent 12c48f87cb
commit 5f853d8aba
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
webhook.go
View File

@ -152,7 +152,7 @@ func hookHandler(w http.ResponseWriter, r *http.Request) {
contentType := r.Header.Get("Content-Type") contentType := r.Header.Get("Content-Type")
if contentType == "application/json" { if strings.HasPrefix(contentType, "application/json") {
decoder := json.NewDecoder(strings.NewReader(string(body))) decoder := json.NewDecoder(strings.NewReader(string(body)))
decoder.UseNumber() decoder.UseNumber()
@ -161,7 +161,7 @@ func hookHandler(w http.ResponseWriter, r *http.Request) {
if err != nil { if err != nil {
log.Printf("error parsing JSON payload %+v\n", err) log.Printf("error parsing JSON payload %+v\n", err)
} }
} else if contentType == "application/x-www-form-urlencoded" { } else if strings.HasPrefix(contentType, "application/x-www-form-urlencoded") {
fd, err := url.ParseQuery(string(body)) fd, err := url.ParseQuery(string(body))
if err != nil { if err != nil {
log.Printf("error parsing form payload %+v\n", err) log.Printf("error parsing form payload %+v\n", err)