-
Notifications
You must be signed in to change notification settings - Fork 900
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
No support for ALPN #858
Comments
This is necessary to connect to AWS IoT on port 443. |
On further review this appears to be quite involved to get working. The PR may take longer than expected. Has this been requested before? |
This was recently added to the Paho C library (and then the C++ and Rust libraries got it for "free"). But I haven't seen a request for the Java library yet. |
Yeah, this would be very helpful to connect to AWS IoT. Unfortunately it looks like it's beyond the level of effort I can expend for the time I was going to commit to it. Is there any way to get this prioritized so Java has feature parity with C/C++/Rust? |
@timmattison Instead of adding ALPN support, do you think it would be appropriate to fix what caused it to break in the first place? See pull request #942 |
Paho doesn't currently allow the use of ALPN. This can be fixed in the SSLNetworkModule class. I will submit a PR for it.
The text was updated successfully, but these errors were encountered: