Fix creating sync subscription for the same path and different methods. Closes #467 (#468)

mthenw · web-flow · commit 42350ff38793 · 2018-06-25T10:16:10.000+02:00
diff --git a/libkv/subscription.go b/libkv/subscription.go
@@ -170,7 +170,7 @@ func (service Service) checkForPathConflict(space, method, path string) error {
 			return err
 		}
 
-		if sub.Type == subscription.TypeSync {
+		if sub.Type == subscription.TypeSync && sub.Method == method {
 			// add existing paths to check
 			tree.AddRoute(sub.Path, FunctionKey{Space: sub.Space, ID: function.ID("")})
 		}

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ func (service Service) checkForPathConflict(space, method, path string) error {`
`170`	`170`	`return err`
`171`	`171`	`}`
`172`	`172`
`173`		`- if sub.Type == subscription.TypeSync {`
	`173`	`+ if sub.Type == subscription.TypeSync && sub.Method == method {`
`174`	`174`	`// add existing paths to check`
`175`	`175`	`tree.AddRoute(sub.Path, FunctionKey{Space: sub.Space, ID: function.ID("")})`
`176`	`176`	`}`