- Issue created by @dmitry.korhov
- 🇨🇭Switzerland berdir Switzerland
This is likely a duplicate of 🐛 Error on install Active , you need to configure a valid endpoint or this happens. I'm arguing in that issue to not have a default endpoint, at least not enforced so it can be disabled.
- 🇵🇱Poland dmitry.korhov Poland, Warsaw
the problem is that we have everything configured and it works flawlessly right after deployment w/o any further changes :)
- 🇨🇭Switzerland berdir Switzerland
deployment as in config deployment?
That makes sense, as modules are enabled first, that triggers a log message, and then the config is imported later.
IMHO still the same situation as the other issue, in that the module should not by default/without any configuration attempt to connect to locaalhost.
- 🇵🇱Poland dmitry.korhov Poland, Warsaw
> deployment as in config deployment?
drush deploy -y
The module should be able to be enabled without requiring configuration and without generating any log messages that could prevent deployment. - 🇨🇭Switzerland berdir Switzerland
> The module should be able to be enabled without requiring configuration and without generating any log messages that could prevent deployment.
I don't disagree, that's exactly what I'm arguing for as well in the other issue. Which IMHO confirms my point that this is a duplicate of the referenced issue.
- First commit to issue fork.
- Merge request !62Issue #3499621: Rework logger to thow a warning if no endpoint is configured → (Open) created by murz
- 🇦🇲Armenia murz Yerevan, Armenia
I reworked the logger to throw a warning if no custom endpoint is configured instead of the error, please check if this fix is suitable.
- 🇨🇭Switzerland berdir Switzerland
Did you see my proposal in the other issue? I think there needs to be an option to disable it completely, and the easiest option would be to require an explicit config. that can be the default config, then for a new install, nothing changes.
And this scenario wouldn't be a problem, it will just not log until the config is in place.