From 0818f3bdf5890ce66b6b82b22bb53ba2976cda9f Mon Sep 17 00:00:00 2001 From: Paul van Staden Date: Sat, 23 Sep 2017 17:13:00 -0700 Subject: [PATCH] Adding TLSv1 support for custom SSL socket factory Summary: This problem tries to solve issue #14244 Implemented a fetch request to a TLSv1 server. It now successfully resolves TLS handshake. Tested same fetch to TLSv1.2 server and still successfully resolves TLS handshake. Closes https://github.com/facebook/react-native/pull/14245 Differential Revision: D5898689 Pulled By: shergin fbshipit-source-id: 8766ebe6909443367651ab868aa5ff62747cd906 --- .../com/facebook/react/modules/network/TLSSocketFactory.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ReactAndroid/src/main/java/com/facebook/react/modules/network/TLSSocketFactory.java b/ReactAndroid/src/main/java/com/facebook/react/modules/network/TLSSocketFactory.java index 53bd9e754..0fd2a2d86 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/modules/network/TLSSocketFactory.java +++ b/ReactAndroid/src/main/java/com/facebook/react/modules/network/TLSSocketFactory.java @@ -71,7 +71,7 @@ public class TLSSocketFactory extends SSLSocketFactory { private Socket enableTLSOnSocket(Socket socket) { if(socket != null && (socket instanceof SSLSocket)) { - ((SSLSocket)socket).setEnabledProtocols(new String[] {"TLSv1.1", "TLSv1.2"}); + ((SSLSocket)socket).setEnabledProtocols(new String[] {"TLSv1", "TLSv1.1", "TLSv1.2"}); } return socket; }