Compare commits
	
		
			7 Commits
		
	
	
		
			v123061811
			...
			4482234e1a
		
	
	| Author | SHA1 | Date | |
|---|---|---|---|
| 4482234e1a | |||
| b5de30f561 | |||
| 70ad5f322c | |||
| d167092c83 | |||
| c4f4bafe85 | |||
| ed06b22a77 | |||
|  | 172362b533 | 
| @@ -1,3 +1,10 @@ | ||||
| **v123061811** | ||||
|  | ||||
| - feat: Added confirmation dialog for disconnect item menu. | ||||
| - Changelog for v123061651 [CI SKIP] | ||||
|  | ||||
| -------------------------------------------------------------------- | ||||
|  | ||||
| **v123061651** | ||||
|  | ||||
| - i18n: Translation update. | ||||
|   | ||||
| @@ -139,9 +139,12 @@ class LoginActivity : AppCompatActivity(), DIAware { | ||||
|  | ||||
|         showProgress(true) | ||||
|  | ||||
|         appSettingsService.updateSelfSigned(binding.selfSigned.isChecked) | ||||
|  | ||||
|         repository.refreshLoginInformation(url, login, password) | ||||
|  | ||||
|         CoroutineScope(Dispatchers.Main).launch { | ||||
|             repository.updateApiInformation() | ||||
|             val result = repository.login() | ||||
|             if (result) { | ||||
|                 val (errorFetching, displaySelfossOnly) = repository.shouldBeSelfossInstance() | ||||
|   | ||||
| @@ -51,6 +51,13 @@ | ||||
|                 android:maxLines="1" | ||||
|                 android:minHeight="48dp" /> | ||||
|  | ||||
|             <com.google.android.material.switchmaterial.SwitchMaterial | ||||
|                 android:id="@+id/selfSigned" | ||||
|                 android:layout_width="match_parent" | ||||
|                 android:layout_height="wrap_content" | ||||
|                 android:text="@string/disable_ssl" | ||||
|                 android:textAlignment="viewStart" /> | ||||
|  | ||||
|             <com.google.android.material.switchmaterial.SwitchMaterial | ||||
|                 android:id="@+id/withLogin" | ||||
|                 android:layout_width="match_parent" | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -1,5 +1,5 @@ | ||||
| <?xml version="1.0" encoding="utf-8"?> | ||||
| <resources xmlns:tools="http://schemas.android.com/tools"> | ||||
| <resources> | ||||
|     <string name="app_name">"Lettore RSS per Selfoss"</string> | ||||
|     <string name="title_activity_login">"Accedi"</string> | ||||
|     <string name="prompt_password">"Password"</string> | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -1,11 +0,0 @@ | ||||
| <?xml version="1.0" encoding="utf-8"?> | ||||
| <resources> | ||||
|     <string name="gdpr_dialog_title">The app does not share any personal data about you.</string> | ||||
|     <string name="gdpr_dialog_message"><![CDATA[Crash reports sending is now enabled. It can be disabled from the settings page. Keep in mind that crash reports are essential for the app development.]]></string> | ||||
|     <string name="crash_toast_text">A crash occured. Sending the details to the developper.</string> | ||||
|     <string name="pref_switch_disable_acra">"Disable automatic bug reporting. "</string> | ||||
|     <string name="menu_home_filter">Filters</string> | ||||
|     <string name="application_selfoss_only">This app only works with a Selfoss instance, and no other RSS feed.</string> | ||||
|     <string name="menu_home_sources">Sources</string> | ||||
|     <string name="update_source">Update source</string> | ||||
| </resources> | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">更新源</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -130,4 +130,5 @@ | ||||
|     <string name="update_source">Update source</string> | ||||
|     <string name="confirm_disconnect_title">Disconnect ?</string> | ||||
|     <string name="confirm_disconnect_description">You will be disconnected from your selfoss instance.</string> | ||||
|     <string name="disable_ssl">Disable SSL</string> | ||||
| </resources> | ||||
|   | ||||
| @@ -6,6 +6,7 @@ | ||||
|     <string name="error_invalid_password">"Password not long enough"</string> | ||||
|     <string name="error_field_required">"Field required"</string> | ||||
|     <string name="prompt_url">"Url"</string> | ||||
|     <string name="disable_ssl">"Disable SSL"</string> | ||||
|     <string name="withLoginSwitch">"Login required ?"</string> | ||||
|     <string name="login_url_problem">"Oops. You may need to add a \"/\" at the end of the url."</string> | ||||
|     <string name="prompt_login">"Username"</string> | ||||
|   | ||||
| @@ -29,13 +29,17 @@ kotlin { | ||||
|     sourceSets { | ||||
|         val commonMain by getting { | ||||
|             dependencies { | ||||
|                 implementation("io.ktor:ktor-client-core:2.1.1") | ||||
|                 implementation("io.ktor:ktor-client-content-negotiation:2.1.1") | ||||
|                 implementation("io.ktor:ktor-serialization-kotlinx-json:2.1.1") | ||||
|                 implementation("io.ktor:ktor-client-logging:2.1.1") | ||||
|                 val ktorVersion = "2.3.2" | ||||
|  | ||||
|                 implementation("io.ktor:ktor-client-core:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-content-negotiation:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-serialization-kotlinx-json:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-logging:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-auth:$ktorVersion") | ||||
|                 implementation("io.ktor:ktor-client-cio:$ktorVersion") | ||||
|                 implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.6.0") | ||||
|                 implementation("io.ktor:ktor-client-auth:2.1.1") | ||||
|                 implementation("org.jsoup:jsoup:1.14.3") | ||||
|  | ||||
|                 implementation("org.jsoup:jsoup:1.15.4") | ||||
|  | ||||
|                 //Dependency Injection | ||||
|                 implementation("org.kodein.di:kodein-di:7.12.0") | ||||
| @@ -58,7 +62,8 @@ kotlin { | ||||
|         } | ||||
|         val androidMain by getting { | ||||
|             dependencies { | ||||
|                 implementation("io.ktor:ktor-client-okhttp:2.1.1") | ||||
|                 implementation("com.squareup.okhttp3:okhttp:4.10.0") | ||||
|                 implementation("io.ktor:ktor-client-okhttp:2.2.4") | ||||
|                 implementation("org.jetbrains.kotlinx:kotlinx-datetime:0.4.0") | ||||
|  | ||||
|                 // Sql | ||||
|   | ||||
| @@ -0,0 +1,17 @@ | ||||
| package bou.amine.apps.readerforselfossv2.rest | ||||
|  | ||||
| import io.ktor.client.engine.cio.CIOEngineConfig | ||||
| import java.security.cert.X509Certificate | ||||
| import javax.net.ssl.X509TrustManager | ||||
|  | ||||
| class NaiveTrustManager : X509TrustManager { | ||||
|     override fun checkClientTrusted(chain: Array<out X509Certificate>?, authType: String?) {} | ||||
|  | ||||
|     override fun checkServerTrusted(chain: Array<out X509Certificate>?, authType: String?) {} | ||||
|  | ||||
|     override fun getAcceptedIssuers(): Array<out X509Certificate> = arrayOf() | ||||
| } | ||||
|  | ||||
| actual fun setupInsecureHTTPEngine(config: CIOEngineConfig) { | ||||
|     config.https.trustManager = NaiveTrustManager() | ||||
| } | ||||
| @@ -5,38 +5,52 @@ import bou.amine.apps.readerforselfossv2.model.StatusAndData | ||||
| import bou.amine.apps.readerforselfossv2.model.SuccessResponse | ||||
| import bou.amine.apps.readerforselfossv2.service.AppSettingsService | ||||
| import io.github.aakira.napier.Napier | ||||
| import io.ktor.client.* | ||||
| import io.ktor.client.plugins.* | ||||
| import io.ktor.client.plugins.auth.providers.* | ||||
| import io.ktor.client.plugins.cache.* | ||||
| import io.ktor.client.plugins.contentnegotiation.* | ||||
| import io.ktor.client.plugins.cookies.* | ||||
| import io.ktor.client.plugins.logging.* | ||||
| import io.ktor.client.request.* | ||||
| import io.ktor.client.statement.* | ||||
| import io.ktor.http.* | ||||
| import io.ktor.serialization.kotlinx.json.* | ||||
| import io.ktor.util.* | ||||
| import io.ktor.utils.io.charsets.* | ||||
| import io.ktor.utils.io.core.* | ||||
| import io.ktor.client.HttpClient | ||||
| import io.ktor.client.engine.cio.CIO | ||||
| import io.ktor.client.engine.cio.CIOEngineConfig | ||||
| import io.ktor.client.plugins.HttpRequestRetry | ||||
| import io.ktor.client.plugins.HttpTimeout | ||||
| import io.ktor.client.plugins.auth.providers.BasicAuthCredentials | ||||
| import io.ktor.client.plugins.cache.HttpCache | ||||
| import io.ktor.client.plugins.contentnegotiation.ContentNegotiation | ||||
| import io.ktor.client.plugins.cookies.HttpCookies | ||||
| import io.ktor.client.plugins.logging.LogLevel | ||||
| import io.ktor.client.plugins.logging.Logger | ||||
| import io.ktor.client.plugins.logging.Logging | ||||
| import io.ktor.client.request.get | ||||
| import io.ktor.client.request.headers | ||||
| import io.ktor.client.request.parameter | ||||
| import io.ktor.client.statement.HttpResponse | ||||
| import io.ktor.http.HttpHeaders | ||||
| import io.ktor.http.HttpStatusCode | ||||
| import io.ktor.http.Parameters | ||||
| import io.ktor.serialization.kotlinx.json.json | ||||
| import io.ktor.util.encodeBase64 | ||||
| import io.ktor.utils.io.charsets.Charsets | ||||
| import io.ktor.utils.io.core.toByteArray | ||||
| import kotlinx.coroutines.CoroutineScope | ||||
| import kotlinx.coroutines.Dispatchers | ||||
| import kotlinx.coroutines.launch | ||||
| import kotlinx.serialization.json.Json | ||||
|  | ||||
| class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
| expect fun setupInsecureHTTPEngine(config: CIOEngineConfig) | ||||
|  | ||||
|     var client = createHttpClient() | ||||
|  | ||||
|     private fun createHttpClient(): HttpClient { | ||||
|         val client = HttpClient { | ||||
| fun createHttpClient( | ||||
|     appSettingsService: AppSettingsService, | ||||
|     api: SelfossApi | ||||
| ) = | ||||
|     HttpClient(CIO) { | ||||
|         if (appSettingsService.getSelfSigned()) { | ||||
|             engine { | ||||
|                 setupInsecureHTTPEngine(this) | ||||
|             } | ||||
|         } | ||||
|         install(ContentNegotiation) { | ||||
|             install(HttpCache) | ||||
|             json(Json { | ||||
|                 prettyPrint = true | ||||
|                 isLenient = true | ||||
|                 ignoreUnknownKeys = true | ||||
|                     explicitNulls = false | ||||
|             }) | ||||
|         } | ||||
|         install(Logging) { | ||||
| @@ -54,13 +68,13 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|         install(HttpRequestRetry) { | ||||
|             maxRetries = 2 | ||||
|             retryIf { _, response -> | ||||
|                     response.status == HttpStatusCode.Forbidden && shouldHavePostLogin() && hasLoginInfo() | ||||
|                 response.status == HttpStatusCode.Forbidden && api.shouldHavePostLogin() && api.hasLoginInfo() | ||||
|             } | ||||
|             modifyRequest { | ||||
|                 Napier.i("Will modify", tag = "HttpSend") | ||||
|                 CoroutineScope(Dispatchers.Main).launch { | ||||
|                     Napier.i("Will login", tag = "HttpSend") | ||||
|                         this@SelfossApi.login() | ||||
|                     api.login() | ||||
|                     Napier.i("Did login", tag = "HttpSend") | ||||
|                 } | ||||
|             } | ||||
| @@ -68,16 +82,16 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|         expectSuccess = false | ||||
|     } | ||||
|  | ||||
| class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|  | ||||
|         return client | ||||
|     } | ||||
|     var client = createHttpClient(appSettingsService, this) | ||||
|  | ||||
|     fun url(path: String) = | ||||
|         "${appSettingsService.getBaseUrl()}$path" | ||||
|  | ||||
|     fun refreshLoginInformation() { | ||||
|         appSettingsService.refreshApiSettings() | ||||
|         client = createHttpClient() | ||||
|         client = createHttpClient(appSettingsService, this) | ||||
|     } | ||||
|  | ||||
|     fun constructBasicAuthValue(credentials: BasicAuthCredentials): String { | ||||
| @@ -88,8 +102,8 @@ class SelfossApi(private val appSettingsService: AppSettingsService) { | ||||
|     } | ||||
|  | ||||
|     // Api version was introduces after the POST login, so when there is a version, it should be available | ||||
|     private fun shouldHavePostLogin() = appSettingsService.getApiVersion() != -1 | ||||
|     private fun hasLoginInfo() = | ||||
|     fun shouldHavePostLogin() = appSettingsService.getApiVersion() != -1 | ||||
|     fun hasLoginInfo() = | ||||
|         appSettingsService.getUserName().isNotEmpty() && appSettingsService.getPassword() | ||||
|             .isNotEmpty() | ||||
|  | ||||
|   | ||||
| @@ -8,6 +8,7 @@ class AppSettingsService(acraSenderServiceProcess: Boolean = false) { | ||||
|     // Api related | ||||
|     private var _apiVersion: Int = -1 | ||||
|     private var _publicAccess: Boolean? = null | ||||
|     private var _selfSigned: Boolean? = null | ||||
|     private var _baseUrl: String = "" | ||||
|     private var _userName: String = "" | ||||
|     private var _basicUserName: String = "" | ||||
| @@ -77,6 +78,22 @@ class AppSettingsService(acraSenderServiceProcess: Boolean = false) { | ||||
|         _publicAccess = settings.getBoolean(API_PUBLIC_ACCESS, false) | ||||
|     } | ||||
|  | ||||
|     fun getSelfSigned(): Boolean { | ||||
|         if (_selfSigned == null) { | ||||
|             refreshSelfSigned() | ||||
|         } | ||||
|         return _selfSigned!! | ||||
|     } | ||||
|  | ||||
|     fun updateSelfSigned(selfSigned: Boolean) { | ||||
|         settings.putBoolean(API_SELF_SIGNED, selfSigned) | ||||
|         refreshSelfSigned() | ||||
|     } | ||||
|  | ||||
|     private fun refreshSelfSigned() { | ||||
|         _selfSigned = settings.getBoolean(API_SELF_SIGNED, false) | ||||
|     } | ||||
|  | ||||
|     fun getBaseUrl(): String { | ||||
|         if (_baseUrl.isEmpty()) { | ||||
|             refreshBaseUrl() | ||||
| @@ -383,6 +400,7 @@ class AppSettingsService(acraSenderServiceProcess: Boolean = false) { | ||||
|         refreshBaseUrl() | ||||
|         refreshApiVersion() | ||||
|         refreshPublicAccess() | ||||
|         refreshSelfSigned() | ||||
|     } | ||||
|  | ||||
|     fun refreshUserSettings() { | ||||
| @@ -468,6 +486,8 @@ class AppSettingsService(acraSenderServiceProcess: Boolean = false) { | ||||
|  | ||||
|         const val API_PUBLIC_ACCESS = "apiPublicAccess" | ||||
|  | ||||
|         const val API_SELF_SIGNED = "apiSelfSigned" | ||||
|  | ||||
|         const val API_ITEMS_NUMBER = "prefer_api_items_number" | ||||
|  | ||||
|         const val API_TIMEOUT = "api_timeout" | ||||
|   | ||||
		Reference in New Issue
	
	Block a user