[WIP] Keep-Alive - Githubissues

teamikl / ninjam-server-js

NINJAM Server

http://teamikl.github.io/ninjam-server-js/

MIT License

3 stars 2 forks source link

[WIP] Keep-Alive #10

Open teamikl opened 10 years ago

teamikl commented 10 years ago

[ ] api doc
[ ] naming for event listener, and protected method.
[ ] move getCurrentTime function to common module

coveralls commented 10 years ago

Coverage decreased (-2.7%) when pulling e809e1a2735179dc5f10e6bdc3e6df1c831cfdcb on keep-alive into 0c56c20917882ab988ba053de3e286bae88e547b on master.

coveralls commented 10 years ago

Coverage decreased (-0.9%) when pulling d8f180b7131e3826e6604745856e93ccf5376445 on keep-alive into 0c56c20917882ab988ba053de3e286bae88e547b on master.

coveralls commented 10 years ago

Coverage remained the same when pulling 4acad13bf6eb90cf62a60f0b3f79e31a02a842bd on keep-alive into 0c56c20917882ab988ba053de3e286bae88e547b on master.

teamikl commented 10 years ago

[x] fix cover task gulp

I should test coverall in local, before commit. It is a good practice to be conscious coverage when write codes.

teamikl commented 10 years ago

Use case scenario: the module should work with any transport protocol. this also make usable as IRC's PING/PONG response. For NINJAM server or client, can be used to implement keep-alive middle-ware.

NOTE: TCP has keep-alive, but this is for check application message layer.