Mattermost-server: Refactor "app/server.go" to use structured logging

Created on 24 Sep 2019  路  3Comments  路  Source: mattermost/mattermost-server

Mattermost is refactoring the logs to be fully structured and we're looking for contributors to help with that effort. This Help Wanted issue is to change the usages of mlog. methods with fmt.Sprintf to proper mlog.Field() calls in file app/server.go.

The expected way to implement it is to open the file app/server.go, look for all calls to mlog.Info, mlog.Debug, etc and in cases where fmt.Sprintf is used to write the log - modify the call to use mlog.Bool, mlog.String, etc to pass the parameters, instead of passing them as part of the format string to Sprintf.

Example: https://github.com/mattermost/mattermost-server/pull/12038


If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-18270

AreTechnical Debt Easy Hacktoberfest Help Wanted PR Exists TecGo

All 3 comments

@mattermod I would love to work on this.

Thanks @Lumexralph :+1:

@hanzei A PR is up for this, feedback will be appreciated.

Was this page helpful?
0 / 5 - 0 ratings