Skip to content
Snippets Groups Projects
Commit 0071ecea authored by Semyon Krestyaninov's avatar Semyon Krestyaninov
Browse files

Merge branch 'fix/PRXS-951-1964-FixZapCustomFields' into feature/PRXS-951-1961-ZapIntergration

parents 03fda18d ee8d2412
No related branches found
No related tags found
No related merge requests found
...@@ -30,32 +30,32 @@ func Event(event string) zap.Field { ...@@ -30,32 +30,32 @@ func Event(event string) zap.Field {
return zap.String("event", event) return zap.String("event", event)
} }
// ObjectID возвращает поле и устанавливает передаваемый аргумент в качестве идентификатора объекта в формате ObjectId. // Object возвращает поле и устанавливает передаваемый аргумент в качестве идентификатора объекта в формате ObjectId.
// Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты. // Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты.
func ObjectID(v any) zap.Field { func Object(v any) zap.Field {
oid, err := id.NewObjectId(v) oid, err := id.NewObjectId(v)
if err != nil { if err != nil {
return zap.Skip() return zap.Skip()
} }
return zap.Reflect("object_id", oid) return zap.Reflect("object", oid)
} }
// CallerID возвращает поле и устанавливает передаваемый аргумент в качестве "вызывающего" в формате ObjectId. // Caller возвращает поле и устанавливает передаваемый аргумент в качестве "вызывающего" в формате ObjectId.
// Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты. // Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты.
func CallerID(v any) zap.Field { func Caller(v any) zap.Field {
oid, err := id.NewObjectId(v) oid, err := id.NewObjectId(v)
if err != nil { if err != nil {
return zap.Skip() return zap.Skip()
} }
return zap.Reflect("caller_id", oid) return zap.Reflect("caller", oid)
} }
// CallerIDFromContext извлекает auth.Principal из контекста и устанавливает его в качестве "вызывающего" в формате ObjectID. // CallerFromContext извлекает auth.Principal из контекста и устанавливает его в качестве "вызывающего" в формате Object.
func CallerIDFromContext(ctx context.Context) zap.Field { func CallerFromContext(ctx context.Context) zap.Field {
if ctx == nil { if ctx == nil {
return zap.Skip() return zap.Skip()
} }
return CallerID(auth.GetPrincipal(ctx)) return Caller(auth.GetPrincipal(ctx))
} }
func Attr(attr any) zap.Field { func Attr(attr any) zap.Field {
......
...@@ -79,10 +79,10 @@ func TestObjectID(t *testing.T) { ...@@ -79,10 +79,10 @@ func TestObjectID(t *testing.T) {
field zap.Field field zap.Field
want zap.Field want zap.Field
}{ }{
{name: "system object", field: ObjectID(item), want: zap.Reflect("object_id", oid)}, {name: "system object", field: Object(item), want: zap.Reflect("object", oid)},
{name: "object id", field: ObjectID(itemId), want: zap.Reflect("object_id", oid)}, {name: "object id", field: Object(itemId), want: zap.Reflect("object", oid)},
{name: "string", field: ObjectID(oid.String()), want: zap.Reflect("object_id", oid)}, {name: "string", field: Object(oid.String()), want: zap.Reflect("object", oid)},
{name: "invalid", field: ObjectID(nil), want: zap.Skip()}, {name: "invalid", field: Object(nil), want: zap.Skip()},
} }
for _, tc := range tests { for _, tc := range tests {
...@@ -109,10 +109,10 @@ func TestCallerID(t *testing.T) { ...@@ -109,10 +109,10 @@ func TestCallerID(t *testing.T) {
field zap.Field field zap.Field
want zap.Field want zap.Field
}{ }{
{name: "system object", field: CallerID(user), want: zap.Reflect("caller_id", oid)}, {name: "system object", field: Caller(user), want: zap.Reflect("caller", oid)},
{name: "object id", field: CallerID(userId), want: zap.Reflect("caller_id", oid)}, {name: "object id", field: Caller(userId), want: zap.Reflect("caller", oid)},
{name: "string", field: CallerID(oid.String()), want: zap.Reflect("caller_id", oid)}, {name: "string", field: Caller(oid.String()), want: zap.Reflect("caller", oid)},
{name: "invalid", field: CallerID(nil), want: zap.Skip()}, {name: "invalid", field: Caller(nil), want: zap.Skip()},
} }
for _, tc := range tests { for _, tc := range tests {
...@@ -135,8 +135,8 @@ func TestCallerIDFromContext(t *testing.T) { ...@@ -135,8 +135,8 @@ func TestCallerIDFromContext(t *testing.T) {
field zap.Field field zap.Field
want zap.Field want zap.Field
}{ }{
{name: "ok", field: CallerIDFromContext(ctx), want: zap.Reflect("caller_id", oid)}, {name: "ok", field: CallerFromContext(ctx), want: zap.Reflect("caller", oid)},
{name: "invalid", field: CallerIDFromContext(nil), want: zap.Skip()}, {name: "invalid", field: CallerFromContext(nil), want: zap.Skip()},
} }
for _, tc := range tests { for _, tc := range tests {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment