Merge pull request #10524 from kobergj/RemoveMBreaker

Remove gobreaker lib
owncloud · Nov 8, 2024 · 305454d · 305454d
2 parents 38151cf + ea726ff
commit 305454d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/changelog/unreleased/remove-mbreaker.md b/changelog/unreleased/remove-mbreaker.md
@@ -0,0 +1,5 @@
+Bugfix: Remove mbreaker
+
+The circuit breaker is not handle correctly and leads therefore to more issues than it solves. We removed it.
+
+https://github.com/owncloud/ocis/pull/10524
diff --git a/ocis-pkg/service/grpc/client.go b/ocis-pkg/service/grpc/client.go
@@ -7,7 +7,6 @@ import (
 	"os"
 
 	mgrpcc "github.com/go-micro/plugins/v4/client/grpc"
-	mbreaker "github.com/go-micro/plugins/v4/wrapper/breaker/gobreaker"
 	mtracer "github.com/go-micro/plugins/v4/wrapper/trace/opentelemetry"
 	"github.com/owncloud/ocis/v2/ocis-pkg/registry"
 	"github.com/owncloud/ocis/v2/ocis-pkg/shared"
@@ -69,7 +68,6 @@ func NewClient(opts ...ClientOption) (client.Client, error) {
 	var tlsConfig *tls.Config
 	cOpts := []client.Option{
 		client.Registry(reg),
-		client.Wrap(mbreaker.NewClientWrapper()),
 		client.Wrap(mtracer.NewClientWrapper(
 			mtracer.WithTraceProvider(options.tp),
 		)),