Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call clearCachedLocationForServer while handling connection error #77

Merged
merged 1 commit into from
Nov 25, 2014

Conversation

wision
Copy link
Contributor

@wision wision commented Nov 25, 2014

No description provided.

@fengmk2
Copy link
Member

fengmk2 commented Nov 25, 2014

close event wont emit?

fengmk2 added a commit that referenced this pull request Nov 25, 2014
Call clearCachedLocationForServer while handling connection error
@fengmk2 fengmk2 merged commit ff99fe8 into alibaba:master Nov 25, 2014
@fengmk2 fengmk2 added the bug label Nov 25, 2014
@fengmk2 fengmk2 self-assigned this Nov 25, 2014
@fengmk2
Copy link
Member

fengmk2 commented Nov 25, 2014

1.2.1 published

@wision wision deleted the error-handle branch November 27, 2014 15:00
@wision
Copy link
Contributor Author

wision commented Nov 27, 2014

That's the strange thing.. it didn't, and this.cachedServers still kept the server in there.

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

Successfully merging this pull request may close these issues.

None yet

2 participants