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

Laravel 8. Undefined index: via #31

Closed
michael-paktolus opened this issue Jul 21, 2021 · 6 comments
Closed

Laravel 8. Undefined index: via #31

michael-paktolus opened this issue Jul 21, 2021 · 6 comments

Comments

@michael-paktolus
Copy link

michael-paktolus commented Jul 21, 2021

I have next error in my laravel.log:
laravel.EMERGENCY: Unable to create configured logger. Using emergency logger. {"exception":"[object] (ErrorException(code: 0): Undefined index: via at \\vendor\\laravel\\framework\\src\\Illuminate\\Log\\LogManager.php:227)
It is require 'via' key: https://laravel.com/docs/8.x/logging#creating-custom-channels-via-factories

@developernaren
Copy link
Contributor

The way this is implemented right now is that it listens to log event and logs to aws. This package does not work as a standalone logging package. I have actually implemented this to work as a driver in this PR. #24
Would you be willing to test this and give me feedback if this works?

@michael-paktolus
Copy link
Author

michael-paktolus commented Jul 27, 2021 via email

@developernaren
Copy link
Contributor

@michael-paktolus thank you for testing.
I made some changes and added a readme too. Can you please test this once more? If everything works, I will release this as v1.0

@michael-paktolus
Copy link
Author

michael-paktolus commented Jul 29, 2021

Hello, @developernaren, thank you for your work. The package working good.
As I see you removed 'disabled' key, but it is still present in config/logging.php

@developernaren
Copy link
Contributor

@michael-paktolus nice catch, removed that, will create a release now.

@developernaren
Copy link
Contributor

released: https://github.com/pagevamp/laravel-cloudwatch-logs/releases/tag/v1.0.1
will close this issue.

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

No branches or pull requests

2 participants