feat(api,cli): delete mongo store

Author: heiytor

api/pkg/echo/handlers/errors.go

@@ -10,7 +10,6 @@ import (
 	routes "github.com/shellhub-io/shellhub/api/routes/errors"
 	"github.com/shellhub-io/shellhub/api/services"
 	"github.com/shellhub-io/shellhub/api/store"
-	"github.com/shellhub-io/shellhub/api/store/mongo"
 	"github.com/shellhub-io/shellhub/pkg/errors"
 )
 
@@ -36,15 +35,6 @@ func NewErrors(reporter *sentry.Client) func(error, echo.Context) {
 		// happens in each case, avoiding the use of else statements, which would make the code more confusing or a big
 		// switch statement, which would make the code less readable.
 
-		// Every Mongo error that isn't mapped as a store error must be reported to Sentry and responded with HTTP
-		// status code 500.
-		if errors.Is(err, mongo.ErrMongo) {
-			report(reporter, err, ctx.Request())
-			ctx.NoContent(http.StatusInternalServerError) //nolint:errcheck
-
-			return
-		}
-
 		// On HTTP errors, anything related to the HTTP protocol, we just return the error code, avoiding a 500 error.
 		var herr *echo.HTTPError
 		if ok := errors.As(err, &herr); ok {

api/services/member.go

@@ -7,7 +7,6 @@ import (
 	"time"
 
 	"github.com/shellhub-io/shellhub/api/store"
-	"github.com/shellhub-io/shellhub/api/store/mongo"
 	"github.com/shellhub-io/shellhub/pkg/api/authorizer"
 	"github.com/shellhub-io/shellhub/pkg/api/requests"
 	"github.com/shellhub-io/shellhub/pkg/clock"
@@ -276,8 +275,6 @@ func (s *service) removeMember(ctx context.Context, ns *models.Namespace, userID
 		switch {
 		case errors.Is(err, store.ErrNoDocuments):
 			return NewErrNamespaceNotFound(ns.TenantID, err)
-		case errors.Is(err, mongo.ErrUserNotFound):
-			return NewErrNamespaceMemberNotFound(userID, err)
 		default:
 			return err
 		}