Re-enable auto-reconnect, after fixing logout.

Authored by dfaure on Feb 13 2020, 11:00 PM.


  • Disconnect websocket in ~DDPClient to avoid auto-reconnect
  • Make logout actually work by returning true in canStart() (obvious typo)

Also, don't clear the password on logout. I want to be able to log in again
without having to type the password again. No idea why this clear() was

Cleanups: remove unused signal logoutDone() and unimplemented method DDPClient::logOut

Test Plan

Logout stays on login page, with password filled in.
Disconnecting from WIFI for a few minutes and then connecting again leads to the server
throwing us out, and ruqola reconnects.

Diff Detail

R865 Ruqola
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
dfaure requested review of this revision.Feb 13 2020, 11:00 PM
dfaure created this revision.
kfunk accepted this revision.Feb 14 2020, 7:45 AM

Looks good to me, but didn't test.

This revision is now accepted and ready to land.Feb 14 2020, 7:45 AM
mlaurent accepted this revision.Feb 14 2020, 8:13 AM
This revision was automatically updated to reflect the committed changes.