[HTTPCORE-525] AuditLog should be a one of strategies rather than the only implementation

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 5.0-beta2
• Component/s: HttpCore NIO
• Labels: volunteers-wanted
• Assignee: Unassigned
• Reporter: Constantine Plotnikov
• Created: 26/Apr/18 22:01
• Updated: 12/Feb/19 09:24
• Resolved: 12/Feb/19 09:24

Description

The method requester.getExceptionLog() returns a list of exceptions that happened during execution. This creates potential memory leak.

I think that this method should be removed, and replaced with some Consumer<Exception> or better Consumer<Throwable>. Or maybe some specialized interface. For backward compatibility, some default impelemtation of this interace could expose itself via getExceptionLog() method. But I think that this is excessive and those who need them as list might plug listener that collects them.

Practially no application needs these exceptions as list. Most application need just to log these exception and forget about them.