From 548bda513c4c6dd43aea2ce489753452bbbcfbee Mon Sep 17 00:00:00 2001
From: Alex Petraky <petraky@perx.ru>
Date: Tue, 2 Apr 2024 12:38:55 +0000
Subject: [PATCH] =?UTF-8?q?=D0=94=D0=BE=D0=B1=D0=B0=D0=B2=D0=BB=D0=B5?=
=?UTF-8?q?=D0=BD=D0=BE=20=D0=BB=D0=BE=D0=B3=D0=B8=D1=80=D0=BE=D0=B2=D0=B0?=
=?UTF-8?q?=D0=BD=D0=B8=D0=B5=20=D1=81=D0=BE=D0=B1=D1=8B=D1=82=D0=B8=D0=B9?=
=?UTF-8?q?=20=D1=81=D0=B5=D1=80=D0=B2=D0=B8=D1=81=D0=B0=20Users?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
---
pkg/users/events.go | 7 ++
pkg/users/middleware/logging_middleware.go | 122 +++++++++++++++++++++
pkg/users/middleware/middleware.go | 2 +-
3 files changed, 130 insertions(+), 1 deletion(-)
create mode 100644 pkg/users/events.go
create mode 100644 pkg/users/middleware/logging_middleware.go
diff --git a/pkg/users/events.go b/pkg/users/events.go
new file mode 100644
index 00000000..c262c017
--- /dev/null
+++ b/pkg/users/events.go
@@ -0,0 +1,7 @@
+package users
+
+const (
+ EventCreate = "users.create"
+ EventUpdate = "users.update"
+ EventDelete = "users.delete"
+)
diff --git a/pkg/users/middleware/logging_middleware.go b/pkg/users/middleware/logging_middleware.go
new file mode 100644
index 00000000..5fb8f82a
--- /dev/null
+++ b/pkg/users/middleware/logging_middleware.go
@@ -0,0 +1,122 @@
+package middleware
+
+import (
+ "context"
+
+ "git.perx.ru/perxis/perxis-go/id"
+ "git.perx.ru/perxis/perxis-go/pkg/options"
+ "git.perx.ru/perxis/perxis-go/pkg/users"
+ logzap "git.perx.ru/perxis/perxis-go/zap"
+ "go.uber.org/zap"
+)
+
+type loggingMiddleware struct {
+ logger *zap.Logger
+ next users.Users
+}
+
+func LoggingMiddleware(logger *zap.Logger) Middleware {
+ return func(next users.Users) users.Users {
+ return &loggingMiddleware{
+ next: next,
+ logger: logger.With(logzap.Component("Users")),
+ }
+ }
+}
+
+func (m *loggingMiddleware) Create(ctx context.Context, create *users.User) (user *users.User, err error) {
+ logger := m.logger.With(
+ logzap.Caller(ctx),
+ logzap.Event(users.EventCreate),
+ logzap.Object(user),
+ )
+
+ user, err = m.next.Create(ctx, create)
+ if err != nil {
+ logger.Error("Failed to create", zap.Error(err), logzap.Channels(logzap.Userlog, logzap.Syslog))
+ return
+ }
+
+ logger.Info("User created", logzap.Channels(logzap.Userlog))
+
+ return user, err
+}
+
+func (m *loggingMiddleware) Get(ctx context.Context, userId string) (user *users.User, err error) {
+ logger := m.logger.With(
+ logzap.Caller(ctx),
+ logzap.Object(id.NewUserId(userId)),
+ )
+
+ user, err = m.next.Get(ctx, userId)
+ if err != nil {
+ logger.Error("Failed to get", zap.Error(err))
+ return
+ }
+
+ return user, err
+}
+
+func (m *loggingMiddleware) Find(ctx context.Context, filter *users.Filter, options *options.FindOptions) (found []*users.User, total int, err error) {
+ logger := m.logger.With(
+ logzap.Caller(ctx),
+ )
+
+ found, total, err = m.next.Find(ctx, filter, options)
+ if err != nil {
+ logger.Error("Failed to find", zap.Error(err))
+ return
+ }
+
+ return found, total, err
+}
+
+func (m *loggingMiddleware) Update(ctx context.Context, update *users.User) (err error) {
+ logger := m.logger.With(
+ logzap.Caller(ctx),
+ logzap.Event(users.EventUpdate),
+ logzap.Object(update),
+ )
+
+ err = m.next.Update(ctx, update)
+ if err != nil {
+ logger.Error("Failed to update", zap.Error(err), logzap.Channels(logzap.Userlog, logzap.Syslog))
+ return
+ }
+
+ logger.Info("User updated", logzap.Channels(logzap.Userlog))
+
+ return err
+}
+
+func (m *loggingMiddleware) Delete(ctx context.Context, userId string) (err error) {
+ logger := m.logger.With(
+ logzap.Caller(ctx),
+ logzap.Event(users.EventDelete),
+ logzap.Object(id.NewUserId(userId)),
+ )
+
+ err = m.next.Delete(ctx, userId)
+ if err != nil {
+ logger.Error("Failed to delete", zap.Error(err), logzap.Channels(logzap.Userlog, logzap.Syslog))
+ return
+ }
+
+ logger.Info("User deleted", logzap.Channels(logzap.Userlog))
+
+ return err
+}
+
+func (m *loggingMiddleware) GetByIdentity(ctx context.Context, identity string) (user *users.User, err error) {
+ logger := m.logger.With(
+ logzap.Caller(ctx),
+ logzap.Object(id.NewUserId(identity)),
+ )
+
+ user, err = m.next.GetByIdentity(ctx, identity)
+ if err != nil {
+ logger.Error("Failed to get by identity", zap.Error(err))
+ return
+ }
+ return user, err
+}
diff --git a/pkg/users/middleware/middleware.go b/pkg/users/middleware/middleware.go
index d9419082..ed64ceed 100644
--- a/pkg/users/middleware/middleware.go
+++ b/pkg/users/middleware/middleware.go
@@ -21,7 +21,7 @@ func WithLog(s users.Users, logger *zap.Logger, log_access bool) users.Users {
if log_access {
s = AccessLoggingMiddleware(logger)(s)
}
- s = ErrorLoggingMiddleware(logger)(s)
+ s = LoggingMiddleware(logger)(s)
s = RecoveringMiddleware(logger)(s)
return s
--
GitLab