-
Notifications
You must be signed in to change notification settings - Fork 63
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
Setting whether to output the initialization log #143
Setting whether to output the initialization log #143
Conversation
This fix may solve some of the issues in #134. The issue also mentions changes to the log output destination and format. Although these points are not supported by this Pull Request, it will prevent log output in a format other than Json from interfering with other programs. /cc @micheleorselli |
@tillkruss @georgeboot |
I like the feature as is, but personally would flip the logic and make logging opt-out using environment variables. It's now technically a breaking change and only few users will care about it. |
Agreed. |
Fixed in cf92542 For details such as the name, I have enabled "Allow edits and access to secrets by maintainers" setting in the Pull Request, so feel free to modify it in any way you like. |
Thanks! |
Hi!
In this pull request, I propose a configurable option to selectively enable logging for the Laravel application initialization process orchestrated by Bref.
Added
log_init
option to the config file published withphp artisan vendor:publish --tag=bref-config
.Motivation
The intention is to streamline the log output to CloudWatch by allowing selective logging, enhancing the readability and utility of the logs.
Currently, logs such as the following are output.
These logs are useful for understanding how Bref is working within the Lambda runtime and whether the configuration of the Laravel application we have deployed is as intended.
However, this verbosity becomes superfluous when multiple cold starts occur in rapid succession, as the same messages are replicated each time.
(For example, when
Bref\LaravelBridge\Queue\QueueHandler
handles a large number of "Push Notifications" sent simultaneously)This option aims to provide flexibility in managing log verbosity based on the developers' needs during different stages of deployment and operation.
Thank you.