-
Notifications
You must be signed in to change notification settings - Fork 27
Description
We have seen multiple reports of a memory leak, and have so far traced it down to the exceptions being raised by the underlying Resolv library when it cannot perform a lookup or a reverse-lookup.
An analysis of the memory dump finds that the exceptions raised by the resolv library and immediately caught by this plugin are not being dereferenced as we would expect them to, and end up flagged as the cause of a subsequent exception, producing an endless chain from the most recent exception to the first.
I believe the root issue to be beyond the scope of this plugin, as the exceptions are rescue'd and immediately allowed to fall out of scope, but that the volume of exceptions raised by this implementation when it is failing many lookups may exacerbate the issue.
Reports:
- v3.0.13 (Logstash 7.5.0)
- v3.0.12 (Logstash 7.1.1)