Merge pull request #10 from go-playground/add-handler-support
Add handler support
This commit is contained in:
+39
@@ -0,0 +1,39 @@
|
||||
language: go
|
||||
go:
|
||||
- 1.7.5
|
||||
- 1.8.1
|
||||
- tip
|
||||
matrix:
|
||||
allow_failures:
|
||||
- go: tip
|
||||
|
||||
notifications:
|
||||
email:
|
||||
recipients: dean.karn@gmail.com
|
||||
on_success: change
|
||||
on_failure: always
|
||||
|
||||
before_install:
|
||||
- go get -u github.com/go-playground/overalls
|
||||
- go get -u github.com/mattn/goveralls
|
||||
- go get -u golang.org/x/tools/cmd/cover
|
||||
- go get -u github.com/golang/lint/golint
|
||||
- go get -u github.com/gordonklaus/ineffassign
|
||||
- mkdir -p $GOPATH/src/gopkg.in
|
||||
- ln -s $GOPATH/src/github.com/$TRAVIS_REPO_SLUG $GOPATH/src/gopkg.in/webhooks.v2
|
||||
- ln -s $GOPATH/src/github.com/$TRAVIS_REPO_SLUG $GOPATH/src/gopkg.in/webhooks.v3
|
||||
|
||||
before_script:
|
||||
- go vet ./...
|
||||
|
||||
script:
|
||||
- gofmt -d -s .
|
||||
- golint ./...
|
||||
- ineffassign ./
|
||||
- go test -v ./...
|
||||
- go test -race
|
||||
|
||||
after_success: |
|
||||
[ $TRAVIS_GO_VERSION = 1.8.1 ] &&
|
||||
overalls -project="github.com/go-playground/webhooks" -covermode=count -ignore=.git,examples -debug &&
|
||||
goveralls -coverprofile=overalls.coverprofile -service travis-ci -repotoken $COVERALLS_TOKEN
|
||||
@@ -1,7 +1,7 @@
|
||||
Library webhooks
|
||||
================
|
||||
<img align="right" src="https://raw.githubusercontent.com/go-playground/webhooks/v3/logo.png">
|
||||
[](https://semaphoreci.com/joeybloggs/webhooks)
|
||||
<img align="right" src="https://raw.githubusercontent.com/go-playground/webhooks/v3/logo.png">
|
||||
[](https://travis-ci.org/go-playground/webhooks)
|
||||
[](https://coveralls.io/github/go-playground/webhooks?branch=v3)
|
||||
[](https://goreportcard.com/report/go-playground/webhooks)
|
||||
[](https://godoc.org/gopkg.in/go-playground/webhooks.v3)
|
||||
|
||||
+25
-11
@@ -35,18 +35,27 @@ type Webhook interface {
|
||||
}
|
||||
|
||||
type server struct {
|
||||
hook Webhook
|
||||
path string
|
||||
hook Webhook
|
||||
path string
|
||||
includePathCheck bool
|
||||
}
|
||||
|
||||
// ProcessPayloadFunc is a common function for payload return values
|
||||
type ProcessPayloadFunc func(payload interface{}, header Header)
|
||||
|
||||
// Handler returns the webhook http.Handler for use in your own Mux implementation
|
||||
func Handler(hook Webhook) http.Handler {
|
||||
return &server{
|
||||
hook: hook,
|
||||
}
|
||||
}
|
||||
|
||||
// Run runs a server
|
||||
func Run(hook Webhook, addr string, path string) error {
|
||||
srv := &server{
|
||||
hook: hook,
|
||||
path: path,
|
||||
hook: hook,
|
||||
path: path,
|
||||
includePathCheck: true,
|
||||
}
|
||||
|
||||
s := &http.Server{Addr: addr, Handler: srv}
|
||||
@@ -58,8 +67,9 @@ func Run(hook Webhook, addr string, path string) error {
|
||||
func RunServer(s *http.Server, hook Webhook, path string) error {
|
||||
|
||||
srv := &server{
|
||||
hook: hook,
|
||||
path: path,
|
||||
hook: hook,
|
||||
path: path,
|
||||
includePathCheck: true,
|
||||
}
|
||||
|
||||
s.Handler = srv
|
||||
@@ -74,8 +84,9 @@ func RunServer(s *http.Server, hook Webhook, path string) error {
|
||||
func RunTLSServer(s *http.Server, hook Webhook, path string) error {
|
||||
|
||||
srv := &server{
|
||||
hook: hook,
|
||||
path: path,
|
||||
hook: hook,
|
||||
path: path,
|
||||
includePathCheck: true,
|
||||
}
|
||||
|
||||
s.Handler = srv
|
||||
@@ -91,9 +102,12 @@ func (s *server) ServeHTTP(w http.ResponseWriter, r *http.Request) {
|
||||
http.Error(w, "405 Method not allowed", http.StatusMethodNotAllowed)
|
||||
return
|
||||
}
|
||||
if r.URL.Path != s.path {
|
||||
http.Error(w, "404 Not found", http.StatusNotFound)
|
||||
return
|
||||
|
||||
if s.includePathCheck {
|
||||
if r.URL.Path != s.path {
|
||||
http.Error(w, "404 Not found", http.StatusNotFound)
|
||||
return
|
||||
}
|
||||
}
|
||||
|
||||
s.hook.ParsePayload(w, r)
|
||||
|
||||
@@ -8,6 +8,8 @@ import (
|
||||
"testing"
|
||||
"time"
|
||||
|
||||
"net/http/httptest"
|
||||
|
||||
. "gopkg.in/go-playground/assert.v1"
|
||||
)
|
||||
|
||||
@@ -47,6 +49,43 @@ func TestMain(m *testing.M) {
|
||||
// teardown
|
||||
}
|
||||
|
||||
func TestHandler(t *testing.T) {
|
||||
|
||||
mux := http.NewServeMux()
|
||||
mux.Handle("/webhooks", Handler(fakeHook))
|
||||
|
||||
s := httptest.NewServer(Handler(fakeHook))
|
||||
defer s.Close()
|
||||
|
||||
payload := "{}"
|
||||
|
||||
req, err := http.NewRequest("POST", s.URL+"/webhooks", bytes.NewBuffer([]byte(payload)))
|
||||
req.Header.Set("Content-Type", "application/json")
|
||||
|
||||
Equal(t, err, nil)
|
||||
|
||||
client := &http.Client{}
|
||||
resp, err := client.Do(req)
|
||||
Equal(t, err, nil)
|
||||
|
||||
defer resp.Body.Close()
|
||||
|
||||
Equal(t, resp.StatusCode, http.StatusOK)
|
||||
|
||||
// Test BAD METHOD
|
||||
req, err = http.NewRequest("GET", s.URL+"/webhooks", bytes.NewBuffer([]byte(payload)))
|
||||
req.Header.Set("Content-Type", "application/json")
|
||||
|
||||
Equal(t, err, nil)
|
||||
|
||||
resp, err = client.Do(req)
|
||||
Equal(t, err, nil)
|
||||
|
||||
defer resp.Body.Close()
|
||||
|
||||
Equal(t, resp.StatusCode, http.StatusMethodNotAllowed)
|
||||
}
|
||||
|
||||
func TestRun(t *testing.T) {
|
||||
|
||||
go Run(fakeHook, "127.0.0.1:3006", "/webhooks")
|
||||
|
||||
Reference in New Issue
Block a user