Sean Fausett
Sean Fausett
@PettitWesley I'm hoping to get this fix into the next 1.9 release. Any ideas how to fix the non-windows builds?
Rebased with recent changes.
Also added the buffer read overrun fixes since I've been testing with those too (with es).
Okay, I'll squash and signoff.
@PettitWesley I only fixed the buffer read overrun in the `opensearch` plugin since it doesn't seem to do bulk operations like the `es` plugin. (Not sure if that's a good...
If this gets approved then I presume I should submit another pull request to master with the same fixes?
@PettitWesley is there anything else I can do to help get this reviewed and merged? 🙏
Missed the [1.9.8 release](https://github.com/fluent/fluent-bit/releases/tag/v1.9.8) too. 😭
Rebased with recent changes. @nokute78 @edsiper PTAL.
@leonardo-albertovich thank you very much for the review! 1. I agree that using `ES_BULK_HEADER` to calculate `required` seems redundant given the 3 subsequent writes to `bulk->ptr` (`i_len` and `j_len` bytes...