Fix cleartext request on pre 5.0 devices

Author: zjn0505

xkcd/build.gradle

@@ -87,6 +87,7 @@ android {
             testProguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-project-test.pro'
             applicationIdSuffix ".debug"
             multiDexEnabled false
+            ext.enableCrashlytics = false
         }
 
         release {

xkcd/src/debug/AndroidManifest.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="utf-8"?>
+<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+    package="xyz.jienan.xkcd">
+
+    <application>
+
+        <meta-data
+            android:name="firebase_crashlytics_collection_enabled"
+            android:value="false" />
+
+    </application>
+
+</manifest>

xkcd/src/main/java/xyz/jienan/xkcd/XkcdApplication.kt

@@ -63,6 +63,8 @@ class XkcdApplication : Application() {
 
     private fun updateLocale() {
         XkcdModel.localizedUrl = resources.getString(R.string.api_xkcd_localization)
-        Crashlytics.setString("locale", Locale.getDefault().toString())
+        if (!BuildConfig.DEBUG) {
+            Crashlytics.setString("locale", Locale.getDefault().toString())
+        }
     }
 }

xkcd/src/main/java/xyz/jienan/xkcd/base/network/NetworkService.kt

@@ -131,7 +131,7 @@ object NetworkService {
 
             client.sslSocketFactory(TLSSocketFactory(), trustManager)
 
-            val spec = ConnectionSpec.Builder(ConnectionSpec.COMPATIBLE_TLS)
+            val tlsSpec = ConnectionSpec.Builder(ConnectionSpec.COMPATIBLE_TLS)
                     .tlsVersions(TlsVersion.TLS_1_2, TlsVersion.TLS_1_1, TlsVersion.TLS_1_0)
                     .cipherSuites(
                             CipherSuite.TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,
@@ -146,7 +146,13 @@ object NetworkService {
                             CipherSuite.TLS_DHE_RSA_WITH_AES_256_CBC_SHA)
                     .build()
 
-            client.connectionSpecs(listOf(spec))
+            val specs = if (Build.VERSION.SDK_INT <= Build.VERSION_CODES.LOLLIPOP) {
+                listOf(tlsSpec, ConnectionSpec.CLEARTEXT)
+            } else {
+                listOf(tlsSpec)
+            }
+
+            client.connectionSpecs(specs)
         } catch (exc: Exception) {
             Timber.e(exc, "Error while setting TLS")
         }