Merge pull request #79 from checkr/zz/fix-http-path

Fix http path context and fix logging noise
checkr · May 28, 2020 · 68fadf5 · 68fadf5
2 parents 6f8bb08 + 69c9266
commit 68fadf5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/handle_actions.go b/handle_actions.go
@@ -50,6 +50,7 @@ func (a ActionSendHTTP) Perform(ctx Context) error {
 	}
 
 	request := gorequest.New().
+		SetDebug(true).
 		SetLogger(newOmLogger(ctx)).
 		CustomMethod(a.Method, urlStr)
 

diff --git a/http.go b/http.go
@@ -42,7 +42,7 @@ func (om *OpenMock) startHTTP() {
 						HTTPContext:     ec,
 						HTTPHeader:      ec.Request().Header,
 						HTTPBody:        string(body),
-						HTTPPath:        ec.Path(),
+						HTTPPath:        ec.Request().URL.String(),
 						HTTPQueryString: ec.QueryString(),
 						om:              om,
 					}

diff --git a/openmock.go b/openmock.go
@@ -2,7 +2,6 @@ package openmock
 
 import (
 	"log"
-	"os"
 
 	"github.com/caarlos0/env"
 	"github.com/goombaio/orderedmap"
@@ -82,8 +81,7 @@ func (om *OpenMock) SetupLogrus() {
 		logrus.WithField("err", err).Fatalf("failed to set logrus level:%s", om.LogLevel)
 	}
 	logrus.SetLevel(l)
-	logrus.SetOutput(os.Stdout)
-	logrus.SetReportCaller(true)
+	logrus.SetFormatter(&logrus.JSONFormatter{})
 }
 
 func (om *OpenMock) SetupRepo() {
@@ -101,7 +99,6 @@ func (om *OpenMock) SetupRepo() {
 func (om *OpenMock) Start() {
 	om.SetupLogrus()
 	om.SetupRepo()
-
 	om.SetRedis()
 
 	err := om.Load()