Remove the config section related to Monolog #406
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I cannot think of any reason for keeping the
sentry.monolog
configuration option as it doesn't offer any benefit over registering the service by hands. Since changing the Monolog configuration is too hard and cannot be done in an automatic way, the user has anyway to edit the configuration files to add the service to the Monolog handlers stack and at that point having to write the service definition imo is the least. Last, registering the service by hands will make it possible to set environment variables as values of the constructor of the handler, rather than us having to provide workarounds to support this feature in the configuration (#384)