Skip to content
This repository has been archived by the owner on Aug 22, 2019. It is now read-only.

Expired Meteor login tokens are not being handled correctly #96

Open
yvdorofeev opened this issue Apr 27, 2017 · 1 comment
Open

Expired Meteor login tokens are not being handled correctly #96

yvdorofeev opened this issue Apr 27, 2017 · 1 comment

Comments

@yvdorofeev
Copy link
Contributor

I've noticed that if meteor login token expires (user doesn't use the app for too long), Meteor.connect() function is not calling the callback.

The problem seems to be inside DDPExtensions.swift, resume(url: String, callback: DDPCallback?) function. Result of self.loginWithToken() call is not being used and, if it's false, callback is never called.

Looking at the code, seems like callback is also not being called if self.loginWithToken()'s callback gets error passed in.

We are currently testing a fix for this issue, I will submit a pull request once we're done.

@derwaldgeist
Copy link

Is this still open? Seems to be fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants