cwh icon indicating copy to clipboard operation
cwh copied to clipboard

Published 20 hours ago • verified publisher icon maxbanton

[BUG] - Declaration of Maxbanton\Cwh\Handler\CloudWatch::write(array $record)...

Open tanerincode opened this issue 1 year ago • 16 comments

Describe the bug Declaration of Maxbanton\Cwh\Handler\CloudWatch::write(array $record): void must be compatible with Monolog\Handler\AbstractProcessingHandler::write(Monolog\LogRecord $record): void

Please tell about your environment:

  • PHP Version: 8.2x
  • Laravel : 10x
  • Operating system (distro): macOS,
  • Application mode:
    • [x] Web app
    • [ ] CLI app
    • [ ] Daemon worker

tanerincode avatar Mar 09 '23 09:03 tanerincode

Yeah I'm having the same issue actually. Same setup too.

maximusblade avatar Mar 09 '23 10:03 maximusblade

Same issue here. Seems to be related to the change of function signature of Monolog\Handler\AbstractProcessingHandler::write() in this commit: https://github.com/Seldaek/monolog/blob/22c8b19358e916c52f1d2170d44e172152de7c25/src/Monolog/Handler/AbstractProcessingHandler.php

markinjapan avatar Mar 10 '23 08:03 markinjapan

Yes, I saw that. but I'm trying to solve this problem in my local workspace. this seems kind of easy but I haven't worked it out yet.

tanerincode avatar Mar 10 '23 08:03 tanerincode

+1

jelledijkstra97 avatar Mar 11 '23 14:03 jelledijkstra97

+1

bassem-shoukry avatar Mar 14 '23 14:03 bassem-shoukry

+1

MarioZ2002 avatar Mar 15 '23 10:03 MarioZ2002

+1

rafaellaurindo avatar Mar 27 '23 20:03 rafaellaurindo

+1

neninja avatar Mar 27 '23 20:03 neninja

any update?

wattnpapa avatar Apr 24 '23 10:04 wattnpapa

any update?

I guess no, I don't think he has enough time for this.

tanerincode avatar Apr 24 '23 10:04 tanerincode

Any news, pretty urgent

stefanrakicfaxi avatar May 03 '23 13:05 stefanrakicfaxi

nope, he is not available, i guess..

tanerincode avatar May 03 '23 13:05 tanerincode

If someone has the capacity to fork this project and manage it I would happily buy them a regular coffee.

I currently do not have the capacity or know how to manage this.

This is an opportunity for someone to take on a worthwhile project.

maximusblade avatar May 12 '23 10:05 maximusblade

I'm not sure why the original developer isn't responding, but I'd be happy to take over until he returns.

For now I've forked this project, merged the feature/php-8.1 branch, and created a pre-release v3.0.0-RC1:

https://github.com/phpnexus/cwh/releases/tag/v3.0.0-RC1

So far it works fine for me, so if anyone would like to try it out please add this to your composer.json require section:

"phpnexus/cwh": "^v3.0.0-RC1@dev"

For now this is just a fork and merge, but I do plan to expand on the documentation more too, especially around the AWS IAM permissions.

markinjapan avatar May 12 '23 16:05 markinjapan

After some testing, I've created a stable release of the feature/php-8.1 branch as v3.0.0:

https://github.com/phpnexus/cwh/releases/tag/v3.0.0

To install, simply run:

composer require phpnexus/cwh

markinjapan avatar May 26 '23 08:05 markinjapan

After some testing, I've created a stable release of the feature/php-8.1 branch as v3.0.0:

https://github.com/phpnexus/cwh/releases/tag/v3.0.0

To install, simply run:

composer require phpnexus/cwh

Great Job, I going to try it hope the above issue has been fixed

amjeed-ay avatar May 31 '23 00:05 amjeed-ay