cluster icon indicating copy to clipboard operation
cluster copied to clipboard

ClusterLogHandler cannot accept all log records

Open wtsergo opened this issue 1 year ago • 0 comments

ClusterLogHandler cannot accept all log records. e.g. in \Amp\Http\Server\Driver\Internal\AbstractHttpDriver::handleInternalServerError you have

$this->logger->error(
  "...",
  [
    ...
    'method' => $request,
    ...
  ]
);

Then ClusterLogHandler cannot send such error context via channel because $request have closures in nested properties. Maybe need to add some kind of filter there? e.g. skip unserializable items?

I can contribute on fix for that. Just let me know how do you think such problem should be handled.

Thanks.

wtsergo avatar May 29 '24 20:05 wtsergo