-
Notifications
You must be signed in to change notification settings - Fork 882
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
android build fixed #65
android build fixed #65
Conversation
The code was actually changed in c6594ae, I assume it doesn't work on Android if there is no custom context specified? Could you describe what happens then? Exception? If so, please show the stacktrace. Thank you. |
HEAD version crashes with following stack:
when I try to create XMPPTCPConnection. According http://stackoverflow.com/questions/11712671/smack-no-response-from-server-not-sure-why-am-i-getting-this-error, 'jks' keystore type is absent on Android, but even if you create your connection like that:
, it still fails with following stack:
|
I've uploaded a new Snapshot of 4.2.0-alpha3 which includes Flowdalic@a9708ef. I think this fixes the issues in a cleaner way then https://github.com/andrey-starodubtsev/Smack/commit/668413e40715b15d6369d65e938122199cbd09f5. Could you test and report back if it works for you? |
It works, however I can see complain about NPE in logcat:
And one still have to set "bks" as default keystore type for Android. |
Thanks for the feedback. Much appreciated. I've uploaded a new snapshot with Flowdalic@39dacaf. This should prevent the NPE. Could you test and report back? |
Fixed merged into master. Thanks for reporting the issue. |
build for android cannot establish TLS connection because lines from
patch were dropped in 640849d. Earlier these changes were added in 7ceb5f0.