flownative / flow-sentry

Integration of Sentry for Neos Flow
MIT License
5 stars 5 forks source link

Logger for Sentry no longer active since PSR-3 introduction #31

Open kdambekalns opened 3 weeks ago

kdambekalns commented 3 weeks ago

The settings in https://github.com/flownative/flow-sentry/blob/f655f0f735dcee0656073a0826e72784910f5830/Configuration/Settings.yaml#L10-L14 are useless, it must be this instead:

Neos:
  Flow:
    psr3:
      Neos\Flow\Log\PsrLoggerFactory:
        systemLogger:
          default:
            class: Flownative\Sentry\Log\SentryFileBackend
kdambekalns commented 2 weeks ago

After thinking about this as well as getting some feedback, this will be solved, but not by #34. Instead, a logger implementation will be written that can be enabled if wanted, and not be active by default.