jolicode / elastically

🔍 JoliCode's Elastica wrapper to bootstrap Elasticsearch PHP integrations
248 stars 36 forks source link

Use specific channel for logs #163

Closed VincentLanglet closed 1 year ago

VincentLanglet commented 1 year ago

Hi @lyrixx @damienalexandre,

I recently saw logs in the app channel coming from Elastica failure inside vendors. I think the app channel should be reserved to the App folder code. Doctrine use a doctrine channel, Symfony use different channels like request, messenger, ...

To me elastically should do the same when passing the logger to the elastica client. This PR fix this by using a custom elastically channel for the logs. (I hope I'm doing it right)

lyrixx commented 1 year ago

Can you rebase please?

lyrixx commented 1 year ago

Thanks 👍🏼

VincentLanglet commented 1 year ago

Thanks 👍🏼

You're welcome. Do you have time for a release ? :)

damienalexandre commented 1 year ago

Released in 1.8.2, thanks @VincentLanglet !