I believe this should maintain the api compatibility, but should take care of some of the issues regarding the connect callback. Also, the reauthentication on failure to connect. However, it attempts to reauthenticate no matter what the return code is, so it might help us down the line to define semantics for other errors.
I also did some fairly standard code cleanup. We should make a TODO to use a logger object.
I believe this should maintain the api compatibility, but should take care of some of the issues regarding the connect callback. Also, the reauthentication on failure to connect. However, it attempts to reauthenticate no matter what the return code is, so it might help us down the line to define semantics for other errors. I also did some fairly standard code cleanup. We should make a TODO to use a logger object.