Skip to content

Commit

Permalink
Send the "callerid" field when connecting.
Browse files Browse the repository at this point in the history
This should be an optional field, but is apparently mandatory in the
core ROS tooling that ships with the Indigo version.

The bug was discovered and fixed by @akru on github.

Fixes #25
  • Loading branch information
acowley committed Nov 5, 2014
1 parent 72d2125 commit fccb0f3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/Ros/Node/RosTcp.hs
Expand Up @@ -70,7 +70,8 @@ negotiatePub ttype md5 sock =
case lookup "tcp_nodelay" connHeader of
Just "1" -> setSocketOption sock NoDelay 0
_ -> return ()
sendAll sock $ genHeader [("md5sum",md5),("type",ttype)]
sendAll sock . genHeader $
[("md5sum",md5), ("type",ttype), ("callerid","roshask")]

-- |Accept new client connections. A new send buffer is allocated for
-- each new client and added to the client list along with an action
Expand Down

0 comments on commit fccb0f3

Please sign in to comment.