Skip to content

#116 monolog update#117

Closed
tanerincode wants to merge 2 commits into
maxbanton:masterfrom
tanerincode:master
Closed

#116 monolog update#117
tanerincode wants to merge 2 commits into
maxbanton:masterfrom
tanerincode:master

Conversation

@tanerincode

Copy link
Copy Markdown

Monologue has made a significant change, it now accepts LogRecords instead of arrays. This PR contains minor changes for the given situation.

  • Other information:
        $config = [
            "group_name" => "my_group_name",
            "stream_name" => "my_stream_name",
            "retention" => 30,
            "sdk" => [
                  'region' => 'eu-east-1',
                  'version' => 'latest',
                  'credentials' => [
                      'key' => getenv('AWS_CLOUDWATCH_KEY'),
                      'secret' => getenv('AWS_CLOUDWATCH_SECRET')
                  ]
            ]
        ];

      $client = new CloudWatchLogsClient($config["sdk"]);

       $handler = new CloudWatch(
            client: $client,
            group: $config["group_name"],
            stream: $config["stream_name"],
            retention: $config["retention"],
            batchSize: 10000,
            tags: [], 
            level: Level::Debug, 
            bubble: true,
            createGroup: false
        );

        $handler->setFormatter(new JsonFormatter());
        $logger = new Logger('here_your_channel_name');
        $logger->pushHandler($handler);

        return $logger;

@vnrmc

vnrmc commented Mar 15, 2023

Copy link
Copy Markdown

Great job, tanerincode! I hope Maksym finds time to approve it :)

@stefanrakicfaxi

Copy link
Copy Markdown

@maxbanton please approve this PR if it's okay for you

@ravewill

ravewill commented Aug 7, 2023

Copy link
Copy Markdown

I'm switching to another CloudWatch logging provider as this is blocking my upgrade to Laravel 10. I'm sure I'm not alone 😞

@SoCooLoveec

Copy link
Copy Markdown

@maxbanton can you please merge this PR and make new release version?

@jandominikair

Copy link
Copy Markdown

@maxbanton Please, do you have any update to merge this PR?

@tanerincode

Copy link
Copy Markdown
Author

@jandominikair i think he is not available for review this pr, someone did this package alternative with this fix, you can find that one in pr's or issues

@maxbanton

Copy link
Copy Markdown
Owner

Thanks @tanerincode for opening this — Monolog 3 + LogRecord support landed in v3.0.0 (#132). Closing as superseded.

@maxbanton maxbanton closed this May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants