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

Added support for console log #1075

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jmordica
Copy link

Details

This feature adds support for console logging when running Bedrock without the -fork startup flag. This prevents the need to have to run an additional Syslog process when Bedrock is deployed in a Docker/Kubernetes environment.

Fixed Issues

Fixes #101

Tests


Internal Testing Reminder: when changing bedrock, please compile auth against your new changes

@jmordica
Copy link
Author

Is there any interest in this PR @tylerkaraszewski @johnmlee101 @quinthar or anyone else who’s active in this project? I also have a couple of others if someone can please take a look? #1077 #1076

@fukawi2
Copy link
Contributor

fukawi2 commented Oct 23, 2024

@tylerkaraszewski This might make sense now that #1906 has been merged?

@tylerkaraszewski
Copy link
Contributor

I think I would do this differently, if we want it, probably with a specific flag to do this, possibly configuring logging to a particular file. We could clean up the command line documentation and options altogether if we want this, there are a fair number that are undocumented and some documented ones that don't do anything.

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

Successfully merging this pull request may close these issues.

Allow application to log to STDOUT
3 participants