diff --git a/log/zap/field.go b/log/zap/field.go index ba27acba6adde67a92967cc5d307c816c41c6d49..acc6932d5fab8b2975d09998557e6ed41197010d 100644 --- a/log/zap/field.go +++ b/log/zap/field.go @@ -30,32 +30,32 @@ func Event(event string) zap.Field { return zap.String("event", event) } -// ObjectID возвращает поле и устанавливает передаваемый аргумент в качестве идентификатора объекта в формате ObjectId. +// Object возвращает поле и устанавливает передаваемый аргумент в качестве идентификатора объекта в формате ObjectId. // Поддерживает типы в формате ObjectId: id.Descriptor, string, map[string]any, системные объекты. -func ObjectID(v any) zap.Field { +func Object(v any) zap.Field { oid, err := id.NewObjectId(v) if err != nil { 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, системные объекты. -func CallerID(v any) zap.Field { +func Caller(v any) zap.Field { oid, err := id.NewObjectId(v) if err != nil { return zap.Skip() } - return zap.Reflect("caller_id", oid) + return zap.Reflect("caller", oid) } -// CallerIDFromContext извлекает auth.Principal из контекста и устанавливает его в качестве "вызывающего" в формате ObjectID. -func CallerIDFromContext(ctx context.Context) zap.Field { +// CallerFromContext извлекает auth.Principal из контекста и устанавливает его в качестве "вызывающего" в формате Object. +func CallerFromContext(ctx context.Context) zap.Field { if ctx == nil { return zap.Skip() } - return CallerID(auth.GetPrincipal(ctx)) + return Caller(auth.GetPrincipal(ctx)) } func Attr(attr any) zap.Field { diff --git a/log/zap/field_test.go b/log/zap/field_test.go index 40d6822b4dac8b0ab59371a6381f3f58ae5d439c..643506bac023d51b25920ae6b5a3c84d511eb342 100644 --- a/log/zap/field_test.go +++ b/log/zap/field_test.go @@ -79,10 +79,10 @@ func TestObjectID(t *testing.T) { field zap.Field want zap.Field }{ - {name: "system object", field: ObjectID(item), want: zap.Reflect("object_id", oid)}, - {name: "object id", field: ObjectID(itemId), want: zap.Reflect("object_id", oid)}, - {name: "string", field: ObjectID(oid.String()), want: zap.Reflect("object_id", oid)}, - {name: "invalid", field: ObjectID(nil), want: zap.Skip()}, + {name: "system object", field: Object(item), want: zap.Reflect("object", oid)}, + {name: "object id", field: Object(itemId), want: zap.Reflect("object", oid)}, + {name: "string", field: Object(oid.String()), want: zap.Reflect("object", oid)}, + {name: "invalid", field: Object(nil), want: zap.Skip()}, } for _, tc := range tests { @@ -109,10 +109,10 @@ func TestCallerID(t *testing.T) { field zap.Field want zap.Field }{ - {name: "system object", field: CallerID(user), want: zap.Reflect("caller_id", oid)}, - {name: "object id", field: CallerID(userId), want: zap.Reflect("caller_id", oid)}, - {name: "string", field: CallerID(oid.String()), want: zap.Reflect("caller_id", oid)}, - {name: "invalid", field: CallerID(nil), want: zap.Skip()}, + {name: "system object", field: Caller(user), want: zap.Reflect("caller", oid)}, + {name: "object id", field: Caller(userId), want: zap.Reflect("caller", oid)}, + {name: "string", field: Caller(oid.String()), want: zap.Reflect("caller", oid)}, + {name: "invalid", field: Caller(nil), want: zap.Skip()}, } for _, tc := range tests { @@ -135,8 +135,8 @@ func TestCallerIDFromContext(t *testing.T) { field zap.Field want zap.Field }{ - {name: "ok", field: CallerIDFromContext(ctx), want: zap.Reflect("caller_id", oid)}, - {name: "invalid", field: CallerIDFromContext(nil), want: zap.Skip()}, + {name: "ok", field: CallerFromContext(ctx), want: zap.Reflect("caller", oid)}, + {name: "invalid", field: CallerFromContext(nil), want: zap.Skip()}, } for _, tc := range tests {