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.

Sign up for GitHub

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

Jump to bottom

add support for remote link detach handling #45

Closed
ppatierno opened this issue Mar 31, 2017 · 0 comments
Closed

add support for remote link detach handling #45

ppatierno opened this issue Mar 31, 2017 · 0 comments

Comments

@ppatierno
Copy link
Member

The ProtonTransport doesn't handle the LINK_REMOTE_DETACH (i.e. the router sends a detach with closed=false due to a lost connection from the client on the other side) so the closeHandler on a ProtonLink isn't raised.
@gemmellr gemmellr changed the title Add handling of LINK_REMOTE_DETACH add support for remote link detach handling Apr 13, 2017
gemmellr added a commit that referenced this issue Apr 14, 2017
@gemmellr
Merge pull request #46 from ppatierno/master
e134c4c 
Update changes from #45 to pass the specific link as result of the AsyncResult when marked succeeded for consistency with oldern existing handlers.
@gemmellr gemmellr mentioned this issue Apr 14, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ppatierno