Issue13 - Closure logic

dcaba commented 6 years ago

Please, forget about commits before Nov 02, 2017 (merging from old branches)

Well, interesting staff I'd say:

Now the program finishes:
- when there's no remaining connections to process/dial (so everything reached the established or error status)
- or when the user presses Ctrl^C or sends a sighub signal
- and it sends a final "report" of the execution, but just with the status of the connections by the end of the execution (replicating what ping does, but without metrics -> different issue)
- Exit status are also coherent (0 only returned if everything was processed successfully)
reallocating also some code in main to a helper package
minor fixes/refactors
we've also exposed dialtimeout as a flag (in seconds... maybe it should be ms?)

chadell commented 6 years ago

Nice PR! just check my comments and the Codacy warnings. Great job 👌

dcaba commented 6 years ago

Let me ignore from Codacy "public" functions I touched but not all them with documentation;)

dachad / tcpgoon