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

support sessionPresent on connack #222

Merged
merged 2 commits into from
Sep 7, 2017

Conversation

yohei1126
Copy link

@yohei1126 yohei1126 commented Jul 12, 2017

Transfer a sessionPresent flag on a CONNACK token from Paho-Java to Paho-Android. fix #200

Signed-off-by: yonishi vivre214@gmail.com

  • You have signed the Eclipse ECA
  • All of your commits have been signed-off with the correct email address (The same one that you used to sign the CLA)
  • If This PR fixes an issue, that you reference the issue below. OR if this is a new issue that you are fixing straight away that you add some Description about the bug and how this will fix it.
  • If this is new functionality, You have added the appropriate Unit tests.

yonishi added 2 commits July 12, 2017 17:04
Signed-off-by: yonishi <vivre214@gmail.com>
Signed-off-by: yonishi <vivre214@gmail.com>
@yohei1126
Copy link
Author

@jpwsutton please check this.

@yohei1126
Copy link
Author

@jpwsutton Could you check it?

1 similar comment
@yohei1126
Copy link
Author

@jpwsutton Could you check it?

@jpwsutton jpwsutton merged commit 640f6c9 into eclipse:develop Sep 7, 2017
@jpwsutton
Copy link
Member

Apologies for the delay in me accepting this PR @yohei1126, thank you very much for taking the time to fix this bug! It will be included in the next SNAPSHOT build.

@yohei1126 yohei1126 deleted the fix_cleansession branch September 7, 2017 22:16
@yohei1126
Copy link
Author

Thanks. When will it be included?

hannesa2 referenced this pull request in hannesa2/paho.mqtt.android Jun 21, 2020
* support sessionPresent on connack

Signed-off-by: yonishi <vivre214@gmail.com>

* fix wrong indentation

Signed-off-by: yonishi <vivre214@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants