pynsq icon indicating copy to clipboard operation
pynsq copied to clipboard

Published 20 hours ago • verified publisher icon nsqio

IDENTIFY still uses short_id and long_id instead of client_id

Open ashleywalsh opened this issue 8 years ago • 2 comments

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

ashleywalsh avatar Jan 27 '16 22:01 ashleywalsh

Ahh, good catch, thanks!

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

mreiferson avatar Jan 28 '16 00:01 mreiferson

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)

ploxiln avatar Dec 11 '17 05:12 ploxiln