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

Make the fastify logger respect the LOG_LEVEL value set in the env #8588

Merged
merged 4 commits into from
Jun 12, 2023

Conversation

orta
Copy link
Contributor

@orta orta commented Jun 12, 2023

The fastify logger wasn't taking the LOG_LEVEL into account without, I think there's a few fastify related logs which can move from debug to trace too, but I'll make those a separate PR

Copy link
Contributor

@jtoar jtoar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @orta! And thanks for removing those lines from the .env.defaults files, always bothered me but kept forgetting about it

@jtoar jtoar merged commit 8ee64cd into redwoodjs:main Jun 12, 2023
5 checks passed
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jun 12, 2023
dac09 added a commit to dac09/redwood that referenced this pull request Jun 13, 2023
…into chore/rename-entry-client

* 'chore/rename-entry-client' of github.com:dac09/redwood:
  Move fastify.logger.infos to fastify.logger.traces (redwoodjs#8590)
  Make the fastify logger respect the LOG_LEVEL value set in the env (redwoodjs#8588)
  feat: Update the SDL types lib (redwoodjs#8586)
  Fix the codegen path for the GraphQL context objcet (redwoodjs#8585)
  chore(deps): update dependency @clerk/types to v3.42.0 (redwoodjs#8584)
  feat(codemod): Add codemod for js to jsx conversion (redwoodjs#8551)
@jtoar jtoar modified the milestones: next-release-patch, v5.3.1 Jun 15, 2023
jtoar added a commit that referenced this pull request Jun 15, 2023
…8588)

* Make the fastify logger respect the LOG_LEVEL value set in the env

* chore: update test project

* chore: add back lines at the end

---------

Co-authored-by: Dominic Saadi <dominiceliassaadi@gmail.com>
jtoar added a commit that referenced this pull request Jun 15, 2023
…8588)

* Make the fastify logger respect the LOG_LEVEL value set in the env

* chore: update test project

* chore: add back lines at the end

---------

Co-authored-by: Dominic Saadi <dominiceliassaadi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix This PR is a fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants