IDENTIFY still uses short_id and long_id instead of client_id

[ashleywalsh]

nsq.async.AsyncConn._on_connect() still sends short_id and long_id fields.

[mreiferson]

Ahh, good catch, thanks!

Need to circle back around to deprecating all of the old functionality across nsqd and official client libs.

[ploxiln]

short_id and long_id are still included for backwards compatibility, but the new fields were added in #169 (just leaving a note for future reference)