cmanaha / python-elasticsearch-logger

Python Elasticsearch handler for the standard python logging framework
Other
232 stars 116 forks source link

set elasticsearch version to 7.17.9 #92

Open thoscilo opened 1 year ago

thoscilo commented 1 year ago

Version 8+ brakes functionality as RequestsHttpConnection is removed.

fixes:

drumadrian commented 1 year ago

Hi @thoscilo

Would you be willing to meet and write up tests so we can merge this request?

thoscilo commented 1 year ago

Hi @drumadrian,

Sure! As this is W/A only and I have local versions fixing the original problem, consider if you would like to merge this PR. (more info about the fix in linked issue)

Merging this PR and releasing new versions may by useful for someone but may not work for those how are using elasticsearch 8.X version.

As I wanted to keep this discussion public, please feel free to write your thoughts about your plan.

thoscilo commented 1 year ago

not needed if https://github.com/cmanaha/python-elasticsearch-logger/pull/93 will be merged.