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

Add LOG_LEVEL Environment Variable #2157

Merged
merged 1 commit into from Jan 23, 2020
Merged

Add LOG_LEVEL Environment Variable #2157

merged 1 commit into from Jan 23, 2020

Conversation

tanner0101
Copy link
Member

@tanner0101 tanner0101 commented Jan 23, 2020 •
edited

Allows log level to be set by exporting the LOG_LEVEL environment variable.

export LOG_LEVEL=trace

This supports the same values that can be passed to --log:

  • trace
  • debug
  • info
  • notice
  • warning
  • error
  • critical

@tanner0101 tanner0101 added the enhancement New feature or request label Jan 23, 2020
@tanner0101 tanner0101 added this to In Progress in Vapor 4 via automation Jan 23, 2020
@tanner0101 tanner0101 changed the title Add LOG_LEVEL environment variable Add LOG_LEVEL Environment Variable Jan 23, 2020
mattpolzin
mattpolzin approved these changes Jan 23, 2020
View reviewed changes
Copy link
Member

@mattpolzin mattpolzin left a comment

Choose a reason for hiding this comment

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

Yessss. This is a great improvement to usability in some environments I am personally very fond of cough Docker cough.

@tanner0101 tanner0101 added the semver-patch Internal changes only label Jan 23, 2020
@tanner0101 tanner0101 merged commit 01deb32 into master Jan 23, 2020
Vapor 4 automation moved this from In Progress to Done Jan 23, 2020
@tanner0101
Copy link
Member Author

These changes are now available in 4.0.0-beta.3.7

@tanner0101 tanner0101 deleted the tn-env-loglevel branch January 23, 2020 03:45
@tanner0101 tanner0101 mentioned this pull request Jan 23, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mattpolzin mattpolzin mattpolzin approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request semver-patch Internal changes only
Projects
Vapor 4
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tanner0101 @mattpolzin