search

gpavlidi / slangouts

Slack up front, Hangouts in the rear.
MIT License
23 stars 5 forks source link

Crash on connect #4

Open davr opened 8 years ago

davr commented 8 years ago

This line is crashing in hangoutsclient.go / StartPolling(): c.lastSync = *getSelfInfo.ResponseHeader.CurrentServerTime I have no experience with golang, so I can't debug it too well. But if I run it under a debugger, I find that this line: getSelfInfo, _ := c.Client.GetSelfInfo() is returning: (*hangouts.GetSelfInfoResponse)(nil), &errors.errorString{s:"proto: bad wiretype for field hangouts.MoodContent.Segment: got wiretype 0, want 2"}

and so when it tries to dereference the nil, it crashes.

davr commented 8 years ago

https://github.com/davr/slangouts/commit/6bf18dbb020b6e8292aabb0e79cddcdd7e2f85fd

This patch lets slangouts run, however there are problems:

  1. Does not get conversation names, I get a bunch of private groups named like hangouts-urg732cd37
  2. Own messages are duplicated -- If I chat from slack, I see my message repeated back at me from the bot
  3. Messages sent from slack, show up underlined, linked, eg if my message is "hello world", in hangouts, users will see it as "hello world" which is a link to "https://hangouts.google.com/webchat/u/0/hello%20world"