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.

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

[Backend]不要なログを削除 #92

Open
gold-kou opened this issue Dec 5, 2021 · 1 comment
Open

[Backend]不要なログを削除 #92

gold-kou opened this issue Dec 5, 2021 · 1 comment
Assignees

Comments

@gold-kou
Copy link
Owner

gold-kou commented Dec 5, 2021

やたらlog.Println でエラーログだしてるけど、HTTPアクセスログだけで十分なものもありそう。
それらを削除する。

@gold-kou gold-kou self-assigned this Dec 5, 2021
@gold-kou
Copy link
Owner Author

contextに setErrMsg/getErrMsgみたいなのを用意して、エラー時にsetErrMsg、loggin_middlewareのnext.ServeHTTP後にgetErrMsgする。そして、AccessLog構造体にErrMsgみたいなフィールド用意すればよさそう。

とはいえ、今の所はcontrollerでlog.Printlnをするというルールを守れていて、ある程度は綺麗さを保てているので、優先度は低い。

@gold-kou gold-kou added the low label Dec 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant