Make sendPacket a protected function

[kenziedeschepper]

Make sendPacket a protected function to allow overrides in classes extending this handler.

[rrajkomar]

Hi,
While the change is simple enough, may I ask what it is you want to achieve that requires changing the method to protected ?
It may be useful for other users and warrant a bigger change in the library codebase that just the method signature.
Thanks.

[kenziedeschepper]

I'm writing an specific extension for Drupal cms. Where I would like to use the symfony process component (Symfony\Component\Process\Process) to handle the api request for sending the log to Loki async. It's based on an existing Drupal module: https://git.drupalcode.org/project/monolog_loki/-/blob/1.x/src/Logger/Handler/LokiHandler.php
The change I would like to make is the following:

protected function sendPacket(array $packet): void {
    $this->connection = FALSE;
    $payload = json_encode($packet, JSON_THROW_ON_ERROR | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES);

    $url = sprintf('%s/loki/api/v1/push', $this->entrypoint);
    $env_vars = [
      'LOKI_URL' => $url,
      'LOKI_AUTH' => implode(':', $this->basicAuth),
      'PAYLOAD' => $payload,
      'CONTENT_LENGTH' => strlen($payload),
    ];

    $process = Process::fromShellCommandline('curl -u "$LOKI_AUTH" -H "Content-Type: application/json" -H "Content-Length: $CONTENT_LENGTH" -d "$PAYLOAD" -X POST $LOKI_URL', NULL, $env_vars);
    $process->start();
  }

[rrajkomar]

Ok.
Is there any specific reason why you don't wan't to use the provided version ?
Maybe we could find a workaround for the issue that prompted this PR.