Merge pull request #51 from Comcast/feature/webpaCommonBump

bumped webpa-common. updated webhook.NewFactory
xmidt-org · Jan 25, 2018 · 8d56143 · 8d56143
2 parents 39a1f31 + a7beb6c
commit 8d56143
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 23 deletions.
diff --git a/src/glide.lock b/src/glide.lock
diff --git a/src/glide.yaml b/src/glide.yaml
@@ -2,6 +2,6 @@ package: .
 homepage: https://github.com/Comcast/tr1d1um
 import:
 - package: github.com/Comcast/webpa-common
-  version: 77f04798b7e3f33770818d7490b5dc3210c1c926
+  version: ff81f093033bfdf7c8d71b36697f9895026dc78b
 - package: github.com/go-ozzo/ozzo-validation
   version: v3.3
diff --git a/src/tr1d1um/tr1d1um.go b/src/tr1d1um/tr1d1um.go
@@ -154,7 +154,7 @@ func tr1d1um(arguments []string) (exitCode int) {
 //baseRouter is pre-configured with the api/v2 prefix path
 //root is the original router used by webHookFactory.Initialize()
 func ConfigureWebHooks(baseRouter *mux.Router, root *mux.Router, preHandler *alice.Chain, v *viper.Viper, logger log.Logger, registry xmetrics.Registry) (*webhook.Factory, int) {
-	webHookFactory, err := webhook.NewFactory(v, &registry)
+	webHookFactory, err := webhook.NewFactory(v)
 
 	if err != nil {
 		fmt.Fprintf(os.Stderr, "Error creating new webHook factory: %s\n", err)