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

Specify default logger in app.CopyAndConfigureTLS #47947

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

rosstimothy
Copy link
Contributor

As some code begins migrating to slog, there will be no logrus logger to provide to this function. Until the transition is complete, allow a nil logger to be provided and use the default logger instead.

As some code begins migrating to slog, there will be no logrus
logger to provide to this function. Until the transition is
complete, allow a nil logger to be provided and use the default
logger instead.
@rosstimothy rosstimothy added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 backport/branch/v16 labels Oct 25, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47947.d3pp5qlev8mo18.amplifyapp.com

@rosstimothy rosstimothy marked this pull request as ready for review October 25, 2024 17:17
@rosstimothy rosstimothy enabled auto-merge October 25, 2024 17:17
@github-actions github-actions bot requested review from fheinecke and r0mant October 25, 2024 17:18
@rosstimothy rosstimothy added this pull request to the merge queue Oct 25, 2024
Merged via the queue into master with commit 4b45b60 Oct 25, 2024
46 checks passed
@rosstimothy rosstimothy deleted the tross/optional_logger branch October 25, 2024 18:11
@public-teleport-github-review-bot

@rosstimothy See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants