Code could benefit from polymorphism

[vrza]

Problem

All transport variants are implemented under a single class (Syslog extends Transport). Behaviour for different transport variants is driven by conditional if/else blocks (see, for example, sendDgram() and connectDgram()). This kind of approach makes the code hard to read and maintain.

Proposed solution

Leverage JavaScript object polymorphism! More specifically, code could be refactored into multiple internal classes implementing the common logging interface, e.g. at least UnixSocketSyslog, UDPSyslog and TCPSyslog. The Syslog constructor could parse the options and instantiate the appropriate subclass, so that existing consumers don't need to be updated.