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

Review logging priority #28

Open
5HT2 opened this issue Jan 16, 2023 · 1 comment
Open

Review logging priority #28

5HT2 opened this issue Jan 16, 2023 · 1 comment
Labels
backend:other Related to the backend code
Milestone

Comments

@5HT2
Copy link
Contributor

5HT2 commented Jan 16, 2023

I want to go over all the logging in the application backend and ensure proper log levels are being used.

Based on this

const (
	// DebugLevel logs are typically voluminous, and are usually disabled in
	// production.
	DebugLevel = zapcore.DebugLevel
	// InfoLevel is the default logging priority.
	InfoLevel = zapcore.InfoLevel
	// WarnLevel logs are more important than Info, but don't need individual
	// human review.
	WarnLevel = zapcore.WarnLevel
	// ErrorLevel logs are high-priority. If an application is running smoothly,
	// it shouldn't generate any error-level logs.
	ErrorLevel = zapcore.ErrorLevel
	// DPanicLevel logs are particularly important errors. In development the
	// logger panics after writing the message.
	DPanicLevel = zapcore.DPanicLevel
	// PanicLevel logs a message, then panics.
	PanicLevel = zapcore.PanicLevel
	// FatalLevel logs a message, then calls os.Exit(1).
	FatalLevel = zapcore.FatalLevel
)
@5HT2 5HT2 converted this from a draft issue Jan 16, 2023
@5HT2 5HT2 added the backend:other Related to the backend code label Jan 17, 2023
@5HT2 5HT2 added this to the 1.0.0 milestone Jan 17, 2023
@5HT2 5HT2 moved this from Todo (within 1 month) to Todo (within 1 week) in tripreporter tasks Jan 24, 2023
@5HT2
Copy link
Contributor Author

5HT2 commented Mar 7, 2023

Moving this to 1.1.0 as it does not affect release.

@5HT2 5HT2 modified the milestones: 1.0.0, 1.1.0 Mar 7, 2023
@5HT2 5HT2 moved this from Todo (within 1 week) to Todo (within 2 weeks) in tripreporter tasks Mar 15, 2023
@settings settings bot removed this from the 1.1.0 milestone Apr 5, 2023
@5HT2 5HT2 added this to the 1.1.0 milestone Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:other Related to the backend code
Projects
Status: Todo (within 2 weeks)
Development

No branches or pull requests

1 participant