Security logging #4

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

Draft

ernestl wants to merge 21 commits into master from ernestl/security-logging

Draft

Security logging #4

many: review improvements

GitHub Actions / woke failed Apr 25, 2026 in 0s

reviewdog [woke] report

reported by reviewdog 🐶

Findings (3)

seclog/seclog_test.go|145 col 1| [error] dummy may be insensitive, use placeholder, sample instead
seclog/seclog_test.go|146 col 39| [error] dummy may be insensitive, use placeholder, sample instead
seclog/seclog_test.go|149 col 57| [error] dummy may be insensitive, use placeholder, sample instead

Filtered Findings (0)

Annotations

Check failure on line 145 in seclog/seclog_test.go

github-actions / woke

[woke] seclog/seclog_test.go#L145

[error] `dummy` may be insensitive, use `placeholder`, `sample` instead

Raw output


            seclog/seclog_test.go:145:1: [error] `dummy` may be insensitive, use `placeholder`, `sample` instead

Check failure on line 146 in seclog/seclog_test.go

github-actions / woke

[woke] seclog/seclog_test.go#L146

[error] `dummy` may be insensitive, use `placeholder`, `sample` instead

Raw output


            seclog/seclog_test.go:146:39: [error] `dummy` may be insensitive, use `placeholder`, `sample` instead

Check failure on line 149 in seclog/seclog_test.go

github-actions / woke

[woke] seclog/seclog_test.go#L149

[error] `dummy` may be insensitive, use `placeholder`, `sample` instead

Raw output


            seclog/seclog_test.go:149:57: [error] `dummy` may be insensitive, use `placeholder`, `sample` instead

View more details on GitHub Actions