scrubber breaks default laravel behavior doccumented in official documentation #45
majchrosoft
started this conversation in
General
Replies: 1 comment
-
is anyone there ? :) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi guys,
I have a small task - extend format of default logger. i must add x-forwarded-for in every log
I do the steps from official laravel documentation
Customizing Monolog for Channels
https://laravel.com/docs/10.x/logging#customizing-monolog-for-channels
It looks like scrubber library breaks default behavior of laravel app.
I found this part of code which seems to be a little odd for me:
vendor/yorcreative/laravel-scrubber/src/Strategies/TapLoader/Loaders/SpecificChannel.php
i think this type of code should always work as extention - modification of config should be forbidden.
Please correct me if i am wrong about that
Probably it is not good idea to throw out existing taps from config/logging.php
For my intuition
logging.php
is much more general
than
scrubber.php
scrubber can extend things, but never modify
Beta Was this translation helpful? Give feedback.
All reactions