Summary

Replace logrus with zap logging

Review Request #2413 — Created March 31, 2023 and submitted March 31, 2023, 5:01 p.m.

Information

Owner

aklitzing@gmail.com

Repository

grim/hgkeeper

Branch

default

Bugs

Depends On

Reviewers

Groups

People

grim

Description

logrus is in maintenance-mode and links to zap and others by itself.
zap has some really good benchmarks and default log output looks a little bit cleaner.

So let's move to zap logging.

Testing Done

Commits

Summary	ID	Author
Replace logrus with zap logging logrus is in maintenance-mode and links to zap and others by itself. zap has some really good benchmarks and default log output looks a little bit cleaner. So let's move to zap logging.	a09411e67804b64f9a4b96408a6e9ac3ca986914	Andre Klitzing

Commits:

	Summary	ID	Author
	Replace logrus with zap logging logrus is in maintenance-mode and links to zap and others by itself. zap has some really good benchmarks and default log output looks a little bit cleaner. So let's move to zap logging.	53a83ad55e718bd3cee2244cfb1402a27f475034	Andre Klitzing
	Replace logrus with zap logging logrus is in maintenance-mode and links to zap and others by itself. zap has some really good benchmarks and default log output looks a little bit cleaner. So let's move to zap logging.	688549967079c983fa728911094495eac9507459	Andre Klitzing

Diff:

Revision 2 (+210 -152)

Show changes

	go.mod
	go.sum
	main.go
	access/access.go
	access/enforcer.go
	access/hgweb.go
	access/users.go
	authorized_keys/command.go
	10 more

Ship it!

```
Ship It!
```
```
Awesome work! Thanks!!
```

Commits:

	Summary	ID	Author
	Replace logrus with zap logging logrus is in maintenance-mode and links to zap and others by itself. zap has some really good benchmarks and default log output looks a little bit cleaner. So let's move to zap logging.	688549967079c983fa728911094495eac9507459	Andre Klitzing
	Replace logrus with zap logging logrus is in maintenance-mode and links to zap and others by itself. zap has some really good benchmarks and default log output looks a little bit cleaner. So let's move to zap logging.	a09411e67804b64f9a4b96408a6e9ac3ca986914	Andre Klitzing

Diff:

Revision 3 (+204 -146)

Show changes

	go.mod
	go.sum
	main.go
	access/access.go
	access/enforcer.go
	access/users.go
	authorized_keys/command.go
	hg/hg.go
	9 more

Ship it!

```
Ship It!
```

Status:: Completed

Status:: Re-opened

Change Summary:: removed the depends on as this was rebased to default
Depends On:: ~~2412 - Use fsnotify to refresh hgweb~~

Status:: Completed