Skip to content

Standard library slog implementation #56

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
Merged
merged 52 commits into from
Aug 6, 2025
Merged

Standard library slog implementation #56

merged 52 commits into from
Aug 6, 2025

Conversation

bengarrett
Copy link
Collaborator

Dropped the zap logger dependency and replaced it with the new standard library slog package.
Also implemented wider coverage of nil pointer checks.

bengarrett added 30 commits July 26, 2025 07:37
@bengarrett
test code
4f1cdc2
@bengarrett
test code
c8edde2
@bengarrett
test code
f50e0ef
@bengarrett
test code
0165600
@bengarrett
print strings
a01faf7
@bengarrett
pre mass removal of methods
00231b3
@bengarrett
currently broken with a panic
ffa795b
@bengarrett
currently broken with a panic
8ce79a3
@bengarrett
more zap to slog replacements, plus removed unused consts from intern…
84827c4 
…al config pkg
@bengarrett
better error handling for nil args
a406b26
@bengarrett
using slog in config/fix[archive] packages
bf02a80
@bengarrett
using slog with the handler but these are untested
95c6247
@bengarrett
new internal panics package to unify the checks of nil pointers as ar…
d9024f3 
…guments
@bengarrett
incomplete slog replacement for html3
e8b9745
@bengarrett
incomplete slog replacement routing
bd20c92
@bengarrett
server with slog startsup
5448271
@bengarrett
breakout of config.go into sep files
0b967f0
@bengarrett
removed checks.go
6345fd2
@bengarrett
printing of startup config is functional
9016bba
@bengarrett
startup now using slog
9649b4b
@bengarrett
removed zaplog pkgs
79303fc
@bengarrett
rename internal/out to internal/logs
c15f611
@bengarrett
unit tests are functional again
8e3fa92
@bengarrett
panics tests for handler/app/
6150535
@bengarrett
panics tests for handlers
04efcbf
@bengarrett
more arguments checks
54e4067
@bengarrett
model args checks
cf443b0
@bengarrett
test removals and fixes
7d2e710
@bengarrett
removed uses of fprint for console output
b7d01e0
@bengarrett
removed the use of os.stdout and stderr
1cd043e
bengarrett added 22 commits August 1, 2025 16:39
@bengarrett
removed hidden debugging statements and replaced with slog
4759e31
@bengarrett
removed unused errs
dbb27db
@bengarrett
removed unused errs for internal
5bfe16a
@bengarrett
removed unused errs for model
431aa6b
@bengarrett
removed unused tools
9697382
@bengarrett
removed unused unused js tools
017f150
@bengarrett
fix gitignore issue for logs pkg
0aa860e
@bengarrett
logging mostly there, still needs some tweaking and save to file
3d4dcd9
@bengarrett
fixed the output of c and a command flags
9682e87
@bengarrett
tweaking info texts
6e39a17
@bengarrett
tweaking repair texts
77a2b02
@bengarrett
middleware is fixed
839a4e6
@bengarrett
placeholder slog-multi that needs to be fleshed out
ec4220b
@bengarrett
slog implementation and our logs package is mostly complete.
329f8b9 
however, unit tests and examples are still needed.
@bengarrett
using root to openfiles.
9ebcfe6
@bengarrett
fix to use abslog config
d313612
@bengarrett
go mod update and comment fixes
07b0ab4
@bengarrett
fixed some potential nil panics
459fea1
@bengarrett
lint corrections.
0e7460c
@bengarrett
more lint fixes but still plenty to go
535d2e6
@bengarrett
finished golangci linter fixes
6e02cd1
@bengarrett
fix for 500 template crash.
e7dfb6d 
removed configurations listing in the configurations.tmpl. it would be
too much work to reprogram it for a novel feature.
@bengarrett bengarrett self-assigned this Aug 6, 2025
@bengarrett bengarrett merged commit a52c96c into main Aug 6, 2025
4 checks passed
@bengarrett bengarrett deleted the slog branch August 6, 2025 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@bengarrett bengarrett

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bengarrett