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.

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

Convert lib/auth to use slog #50577

Merged
merged 22 commits into from
Jan 6, 2025
Merged

Convert lib/auth to use slog #50577

merged 22 commits into from
Jan 6, 2025

Conversation

rosstimothy
Copy link
Contributor

No description provided.

@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label Dec 26, 2024
@rosstimothy
Convert auth_with_roles to use slog
2ca18a9
@rosstimothy
allow benchmark tests to log
6f04a56
@rosstimothy
remove last use of logrus in github.go
2188b73
@rosstimothy
remove last use of logrus in rotate.go
e8ebda7
@rosstimothy
remove auth package logrus logger
3c90823
@rosstimothy
convert auth.go to exclusively use slog
475ea24
@rosstimothy
convert db.go to exclusively use slog
a507ec8
@rosstimothy
convert methods.go to exclusively use slog
85342cf
@rosstimothy
convert password.go to exclusively use slog
af955cf
@rosstimothy
convert join_ec2.go to exclusively use slog
1f8cf97
@rosstimothy
convert join_iam.go to exclusively use slog
8963371
@rosstimothy
convert kube.go to exclusively use slog
7cdf744
@rosstimothy
convert oidc.go to exclusively use slog
63a2f3c
@rosstimothy
convert saml.go to exclusively use slog
7d03e16
@rosstimothy
convert sso_diag_context.go to exclusively use slog
75703e9
@rosstimothy
convert transport_credentials.go to exclusively use slog
421aa8a
@rosstimothy
convert trustedcluster.go to exclusively use slog
1ef4f17
@rosstimothy
convert usertoken.go to exclusively use slog
aab97b3
@rosstimothy
convert user.go to exclusively use slog
3b01850
@rosstimothy
remove logrus import
a690266
@rosstimothy
fix: pass in context
256dacc
@rosstimothy rosstimothy marked this pull request as ready for review December 30, 2024 17:21
@rosstimothy rosstimothy added this pull request to the merge queue Jan 6, 2025
Merged via the queue into master with commit ec2718b Jan 6, 2025
44 checks passed
@rosstimothy rosstimothy deleted the tross/auth_slog branch January 6, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codingllama codingllama codingllama approved these changes

@zmb3 zmb3 zmb3 approved these changes

Assignees
No one assigned
Labels
no-changelog Indicates that a PR does not require a changelog entry size/md
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rosstimothy @zmb3 @codingllama