From 2697988bbc7c7bc4f9fe88038c623abbd54e9571 Mon Sep 17 00:00:00 2001 From: Chris Danis Date: Fri, 29 Aug 2025 12:57:22 -0400 Subject: [PATCH 1/3] fix: add User-Agent to NetworkingModule http client --- app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt b/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt index 2539db3128..f4933e4a8e 100644 --- a/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt +++ b/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt @@ -7,6 +7,7 @@ import dagger.Provides import fr.free.nrw.commons.BetaConstants import fr.free.nrw.commons.BuildConfig import fr.free.nrw.commons.OkHttpConnectionFactory +import fr.free.nrw.commons.OkHttpConnectionFactory.CommonHeaderRequestInterceptor import fr.free.nrw.commons.actions.PageEditClient import fr.free.nrw.commons.actions.PageEditInterface import fr.free.nrw.commons.actions.ThanksInterface @@ -60,6 +61,7 @@ class NetworkingModule { .connectTimeout(120, TimeUnit.SECONDS) .writeTimeout(120, TimeUnit.SECONDS) .addInterceptor(httpLoggingInterceptor) + .addInterceptor(CommonHeaderRequestInterceptor()) .readTimeout(120, TimeUnit.SECONDS) .cache(Cache(File(context.cacheDir, "okHttpCache"), OK_HTTP_CACHE_SIZE)) .build() From 11b801d152c00807fc4fc85a740d087de264a254 Mon Sep 17 00:00:00 2001 From: Chris Danis Date: Fri, 29 Aug 2025 13:25:12 -0400 Subject: [PATCH 2/3] CommonHeaderRequestInterceptor: publicize --- .../main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt b/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt index d15c72f570..b3ba9e5e75 100644 --- a/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt +++ b/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt @@ -50,7 +50,7 @@ object OkHttpConnectionFactory { } } -private class CommonHeaderRequestInterceptor : Interceptor { +public class CommonHeaderRequestInterceptor : Interceptor { @Throws(IOException::class) override fun intercept(chain: Interceptor.Chain): Response { val request = chain.request().newBuilder() From 8515c55c325331bf644f7763314e5b33ea91719a Mon Sep 17 00:00:00 2001 From: Chris Danis Date: Fri, 29 Aug 2025 14:08:58 -0400 Subject: [PATCH 3/3] fix import & style --- .../main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt | 2 +- app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt b/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt index b3ba9e5e75..1c28d5fe4b 100644 --- a/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt +++ b/app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.kt @@ -50,7 +50,7 @@ object OkHttpConnectionFactory { } } -public class CommonHeaderRequestInterceptor : Interceptor { +class CommonHeaderRequestInterceptor : Interceptor { @Throws(IOException::class) override fun intercept(chain: Interceptor.Chain): Response { val request = chain.request().newBuilder() diff --git a/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt b/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt index f4933e4a8e..9246ff303a 100644 --- a/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt +++ b/app/src/main/java/fr/free/nrw/commons/di/NetworkingModule.kt @@ -7,7 +7,7 @@ import dagger.Provides import fr.free.nrw.commons.BetaConstants import fr.free.nrw.commons.BuildConfig import fr.free.nrw.commons.OkHttpConnectionFactory -import fr.free.nrw.commons.OkHttpConnectionFactory.CommonHeaderRequestInterceptor +import fr.free.nrw.commons.CommonHeaderRequestInterceptor import fr.free.nrw.commons.actions.PageEditClient import fr.free.nrw.commons.actions.PageEditInterface import fr.free.nrw.commons.actions.ThanksInterface