chore(server): add logs on internal error

reearth · Jun 27, 2023 · 2b616d8 · 2b616d8
1 parent 1b53dd1
commit 2b616d8
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 2 deletions.
diff --git a/server/internal/adapter/gql/loader_plugin.go b/server/internal/adapter/gql/loader_plugin.go
@@ -6,6 +6,7 @@ import (
 	"github.com/reearth/reearth/server/internal/adapter/gql/gqldataloader"
 	"github.com/reearth/reearth/server/internal/adapter/gql/gqlmodel"
 	"github.com/reearth/reearth/server/internal/usecase/interfaces"
+	"github.com/reearth/reearth/server/pkg/log"
 	"github.com/reearth/reearthx/util"
 )
 
@@ -23,7 +24,11 @@ func (c *PluginLoader) Fetch(ctx context.Context, ids []gqlmodel.ID) ([]*gqlmode
 		return nil, []error{err}
 	}
 
-	res, err := c.usecase.Fetch(ctx, ids2, getOperator(ctx))
+	op := getOperator(ctx)
+	log.Infof("TEMP: PluginLoader.Fetch op: %#v", op)
+	log.Infof("TEMP: PluginLoader.Fetch ids: %v", ids)
+
+	res, err := c.usecase.Fetch(ctx, ids2, op)
 	if err != nil {
 		return nil, []error{err}
 	}

diff --git a/server/internal/adapter/gql/loader_property.go b/server/internal/adapter/gql/loader_property.go
@@ -7,6 +7,7 @@ import (
 	"github.com/reearth/reearth/server/internal/adapter/gql/gqlmodel"
 	"github.com/reearth/reearth/server/internal/usecase/interfaces"
 	"github.com/reearth/reearth/server/pkg/id"
+	"github.com/reearth/reearth/server/pkg/log"
 	"github.com/reearth/reearthx/util"
 )
 
@@ -24,7 +25,11 @@ func (c *PropertyLoader) Fetch(ctx context.Context, ids []gqlmodel.ID) ([]*gqlmo
 		return nil, []error{err}
 	}
 
-	res, err := c.usecase.Fetch(ctx, ids2, getOperator(ctx))
+	op := getOperator(ctx)
+	log.Infof("TEMP: PropertyLoader.Fetch op: %#v", op)
+	log.Infof("TEMP: PropertyLoader.Fetch ids: %v", ids)
+
+	res, err := c.usecase.Fetch(ctx, ids2, op)
 	if err != nil {
 		return nil, []error{err}
 	}

diff --git a/server/internal/app/app.go b/server/internal/app/app.go
@@ -139,6 +139,7 @@ func initEcho(ctx context.Context, cfg *ServerConfig) *echo.Echo {
 
 func errorHandler(next func(error, echo.Context)) func(error, echo.Context) {
 	return func(err error, c echo.Context) {
+		ctx := c.Request().Context()
 		if c.Response().Committed {
 			return
 		}
@@ -147,6 +148,15 @@ func errorHandler(next func(error, echo.Context)) func(error, echo.Context) {
 			err = rerror.ErrNotFound
 		}
 
+		if err := rerror.UnwrapErrInternal(err); err != nil {
+			au := adapter.GetAuthInfo(ctx)
+			u := adapter.User(ctx)
+			op := adapter.Operator(ctx)
+			c.Echo().Logger.Errorf("AUTH: %#v", au)
+			c.Echo().Logger.Errorf("USER: %#v", u)
+			c.Echo().Logger.Errorf("OP: %#v", op)
+		}
+
 		code, msg := errorMessage(err, func(f string, args ...interface{}) {
 			c.Echo().Logger.Errorf(f, args...)
 		})

diff --git a/server/internal/infrastructure/mongo/plugin.go b/server/internal/infrastructure/mongo/plugin.go
@@ -11,6 +11,7 @@ import (
 	"github.com/reearth/reearth/server/pkg/builtin"
 	"github.com/reearth/reearth/server/pkg/id"
 	"github.com/reearth/reearth/server/pkg/plugin"
+	"github.com/reearth/reearthx/log"
 	"github.com/reearth/reearthx/mongox"
 	"github.com/reearth/reearthx/rerror"
 )
@@ -76,12 +77,16 @@ func (r *Plugin) FindByIDs(ctx context.Context, ids []id.PluginID) ([]*plugin.Pl
 	var err error
 
 	if len(ids2) > 0 {
+		log.Infof("TEMP: plugin mongo find by ids %v", ids2)
+
 		res, err = r.find(ctx, bson.M{
 			"id": bson.M{"$in": id.PluginIDsToStrings(ids2)},
 		})
 		if err != nil {
 			return nil, err
 		}
+
+		log.Infof("TEMP: plugin mongo find by ids OK")
 	}
 
 	return res.Concat(b.List()).MapToIDs(ids), nil

diff --git a/server/internal/infrastructure/mongo/property.go b/server/internal/infrastructure/mongo/property.go
@@ -7,6 +7,7 @@ import (
 	"github.com/reearth/reearth/server/internal/usecase/repo"
 	"github.com/reearth/reearth/server/pkg/id"
 	"github.com/reearth/reearth/server/pkg/property"
+	"github.com/reearth/reearthx/log"
 	"github.com/reearth/reearthx/mongox"
 	"github.com/reearth/reearthx/rerror"
 	"go.mongodb.org/mongo-driver/bson"
@@ -68,6 +69,8 @@ func (r *Property) FindByIDs(ctx context.Context, ids id.PropertyIDList) (proper
 		return nil, nil
 	}
 
+	log.Infof("TEMP: property mongo find by ids %v", ids)
+
 	filter := bson.M{
 		"id": bson.M{
 			"$in": ids.Strings(),
@@ -77,6 +80,9 @@ func (r *Property) FindByIDs(ctx context.Context, ids id.PropertyIDList) (proper
 	if err != nil {
 		return nil, err
 	}
+
+	log.Infof("TEMP: property mongo find by ids OK")
+
 	return filterProperties(ids, res), nil
 }