Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
1 of 3 tasks
tanerincode opened this issue Mar 9, 2023 · 16 comments
Open
1 of 3 tasks
Assignees
Labels

Comments

@tanerincode
Copy link

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:
    • Web app
    • CLI app
    • Daemon worker
@tanerincode tanerincode added the bug label Mar 9, 2023
@tanerincode tanerincode changed the title [BUG] [BUG] - Declaration of Maxbanton\Cwh\Handler\CloudWatch::write(array $record)... Mar 9, 2023
@maximusblade
Copy link

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

@markinjapan
Copy link

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

@tanerincode
Copy link
Author

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.

@jelledijkstra97
Copy link

+1

4 similar comments
@bassem-shoukry
Copy link

+1

@MarioZ2002
Copy link

+1

@rafaellaurindo
Copy link

+1

@neninja
Copy link

neninja commented Mar 27, 2023

+1

@wattnpapa
Copy link

any update?

@tanerincode
Copy link
Author

tanerincode commented Apr 24, 2023

any update?

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

@stefanrakicfaxi
Copy link

Any news, pretty urgent

@tanerincode
Copy link
Author

nope, he is not available, i guess..

@maximusblade
Copy link

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.

@markinjapan
Copy link

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
Copy link

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

@amjeed-ay
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests