Deprecate ILoggingAwareException
authorTim Düsterhus <duesterhus@woltlab.com>
Tue, 12 Oct 2021 13:47:56 +0000 (15:47 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Tue, 12 Oct 2021 13:53:34 +0000 (15:53 +0200)
commit55343e879a9546d30539d7c746b2de22733469be
tree8a85e9f29068e65581f66fd6da3cd7dc75b1b2a0
parentddd2a38ef905f57a0aa45333423f3b3e21215905
Deprecate ILoggingAwareException

The `finalizeLog()` method was initially added to support
com.woltlab.wcf.elasticSearch, as it logged the full - possibly huge -
Elasticsearch response, allowing it to log the response into a separate log
file.

This came with severe usability issues, as this log file is not readily
available from the ACP.

The Elasticsearch package was completely cleaned up, relying on the regular
Guzzle exceptions, and errors during JSON decoding no longer include the full
JSON.

Letting the Exception know that they've been logged is a layering violation
that will not play along nicely with #4342. The current method signature also
is pretty much limited to logging into files only.

Deprecate the interface, now that the only known user is gone.
wcfsetup/install/files/lib/core.functions.php
wcfsetup/install/files/lib/system/exception/ILoggingAwareException.class.php