Fix: Optional Flow exception dump files for SentryStorage (storageLogging)

[Schuppel1]

Problem

When using flownative/sentry as Neos.Flow.log.throwables.storageClass, throwables are reported to Sentry, but Flow/Neos no longer writes exception dump files to Data/Logs/Exceptions/*.txt. This breaks the developer experience (“For a full stacktrace, open …”) because the referenced file may not exist.

Solution

Flownative\Sentry\Log\SentryStorage now optionally writes Flow exception dump files by delegating to Neos\Flow\Log\ThrowableStorage\FileStorage, while still capturing the throwable to Sentry.

New setting (default: false)

Flownative:
  Sentry:
    storageLogging: false

Changes

• Classes/Log/SentryStorage.php

• Add optional FileStorage dump writing controlled by Flownative.Sentry.storageLogging
• Keep Sentry capture (best-effort)

• Configuration/Settings.yaml

• Add storageLogging: false default

• README.md

• Document storageLogging

Testing

• Configure SentryStorage as throwable storage and trigger an exception (storageLogging: true):

1. Expected: Sentry event is created
2. Expected (default): dump file is created under Data/Logs/Exceptions/*.txt
3. Tested with no Sentry DSN

• Set storageLogging: false:

1. Expected: Sentry event yes, dump file no
2. Tested with no Sentry DSN

Environment

• PHP: 8.2
• flownative/sentry: 3.2.0
• sentry/sentry: ^4.0
• Neos Flow: 9.1

[kdambekalns]

Thanks! I'll have a look!

I'll need to dig into my memory, but I would guess it wasn't an active decision to no longer write those files at all.