Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The in.Content of the custom log handler is lost #3200

Closed
hailaz opened this issue Dec 13, 2023 · 6 comments
Closed

The in.Content of the custom log handler is lost #3200

hailaz opened this issue Dec 13, 2023 · 6 comments
Assignees
@gqcn
Labels
bug It is confirmed a bug, but don't worry, we'll handle it. done This issue is done, which may be release in next version.

Comments

@hailaz
Copy link
Member

hailaz commented Dec 13, 2023 •
edited
Loading

因为这个提交删除了相关处理
https://github.com/gogf/gf/pull/2919/files#diff-0950fd3e63cfb4b28566d3c8332c953272a247a2ac36d50cf6b4e299f1aea419L209

测试代码
image

gf/os/glog/glog_z_unit_logger_handler_test.go

Line 23 in 887803e

arrayForHandlerTest1.Append(input.String(false))

@hailaz hailaz added the bug It is confirmed a bug, but don't worry, we'll handle it. label Dec 13, 2023
@Issues-translate-bot Issues-translate-bot changed the title 自定义日志handler的in.Content丢失 The in.Content of the custom log handler is lost Dec 13, 2023
@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

Because this submission deletes related processing
https://github.com/gogf/gf/pull/2919/files#diff-0950fd3e63cfb4b28566d3c8332c953272a247a2ac36d50cf6b4e299f1aea419L209

@hailaz
Copy link
Member Author

hailaz commented Dec 19, 2023

@gqcn 有空看看

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

@gqcn Have a look when you have time

@gqcn
Copy link
Member

gqcn commented Dec 20, 2023

@hailaz OK, let me have a check.

@l12ab
Copy link

l12ab commented Jan 30, 2024

遇到了同样的情况，in.Content为空

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

Encountered the same situation, in.Content is empty

gqcn added a commit that referenced this issue Apr 11, 2024
@gqcn
fix: #3200 empty content in glog handler
0b194d6
@gqcn gqcn added the done This issue is done, which may be release in next version. label Apr 11, 2024
@gqcn gqcn closed this as completed Apr 11, 2024
gqcn added a commit that referenced this issue Apr 25, 2024
@gqcn
fix: #3200 empty content in glog handler after structure logging feat…
cc98e57 
…ure supported (#3475)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gqcn gqcn

Labels
bug It is confirmed a bug, but don't worry, we'll handle it. done This issue is done, which may be release in next version.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@l12ab @gqcn @hailaz @Issues-translate-bot