Allow sync manually, instead force sync at ever login

Sync moved to helper, can be called from login and manually from api
one of the benefits of moving this to a separate function is that we can do some more complex syncing, and unit test. For this reason I'd like to keep this function syncronous and remove the go routine.

Instead we could do `go helper.SyncUser` from the handlers.