From 3167c7a5d2333d94b81edc6d6b567bd4facad1ed Mon Sep 17 00:00:00 2001
From: ByteHamster <info@bytehamster.com>
Date: Mon, 16 Mar 2020 01:09:32 +0100
Subject: Always add UserAgent to requests

---
 .../core/glide/ChapterImageModelLoader.java          |  1 -
 .../antennapod/core/gpoddernet/GpodnetService.java   |  7 +++----
 .../core/service/UserAgentInterceptor.java           | 20 ++++++++++++++++++++
 .../core/service/download/AntennapodHttpClient.java  |  2 ++
 .../core/service/download/HttpDownloader.java        |  3 +--
 5 files changed, 26 insertions(+), 7 deletions(-)
 create mode 100644 core/src/main/java/de/danoeh/antennapod/core/service/UserAgentInterceptor.java

(limited to 'core/src/main/java')

diff --git a/core/src/main/java/de/danoeh/antennapod/core/glide/ChapterImageModelLoader.java b/core/src/main/java/de/danoeh/antennapod/core/glide/ChapterImageModelLoader.java
index bc0a06a07..36da11eca 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/glide/ChapterImageModelLoader.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/glide/ChapterImageModelLoader.java
@@ -71,7 +71,6 @@ public final class ChapterImageModelLoader implements ModelLoader<EmbeddedChapte
                     callback.onDataReady(ByteBuffer.wrap(imageContent));
                 } else {
                     Request.Builder httpReq = new Request.Builder();
-                    httpReq.header("User-Agent", ClientConfig.USER_AGENT);
                     // Skipping would download the whole file
                     httpReq.header("Range", "bytes=" + image.getPosition()
                             + "-" + (image.getPosition() + image.getLength()));
diff --git a/core/src/main/java/de/danoeh/antennapod/core/gpoddernet/GpodnetService.java b/core/src/main/java/de/danoeh/antennapod/core/gpoddernet/GpodnetService.java
index 9ee8c0fc1..97931237a 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/gpoddernet/GpodnetService.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/gpoddernet/GpodnetService.java
@@ -554,9 +554,8 @@ public class GpodnetService {
         }.start();
     }
 
-    private String executeRequest(@NonNull Request.Builder requestB)
-            throws GpodnetServiceException {
-        Request request = requestB.header("User-Agent", ClientConfig.USER_AGENT).build();
+    private String executeRequest(@NonNull Request.Builder requestB) throws GpodnetServiceException {
+        Request request = requestB.build();
         String responseString = null;
         Response response = null;
         ResponseBody body = null;
@@ -584,7 +583,7 @@ public class GpodnetService {
                     "request and credentials must not be null");
         }
 
-        Request request = requestB.header("User-Agent", ClientConfig.USER_AGENT).build();
+        Request request = requestB.build();
         String result = null;
         ResponseBody body = null;
         try {
diff --git a/core/src/main/java/de/danoeh/antennapod/core/service/UserAgentInterceptor.java b/core/src/main/java/de/danoeh/antennapod/core/service/UserAgentInterceptor.java
new file mode 100644
index 000000000..5fcf8317d
--- /dev/null
+++ b/core/src/main/java/de/danoeh/antennapod/core/service/UserAgentInterceptor.java
@@ -0,0 +1,20 @@
+package de.danoeh.antennapod.core.service;
+
+import de.danoeh.antennapod.core.ClientConfig;
+import okhttp3.Interceptor;
+import okhttp3.Response;
+
+import java.io.IOException;
+
+public class UserAgentInterceptor implements Interceptor {
+
+    public UserAgentInterceptor() {
+    }
+
+    @Override
+    public Response intercept(Chain chain) throws IOException {
+        return chain.proceed(chain.request().newBuilder()
+                .header("User-Agent", ClientConfig.USER_AGENT)
+                .build());
+    }
+}
diff --git a/core/src/main/java/de/danoeh/antennapod/core/service/download/AntennapodHttpClient.java b/core/src/main/java/de/danoeh/antennapod/core/service/download/AntennapodHttpClient.java
index ee0f8bd95..e0c23bdac 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/service/download/AntennapodHttpClient.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/service/download/AntennapodHttpClient.java
@@ -32,6 +32,7 @@ import javax.net.ssl.TrustManagerFactory;
 import javax.net.ssl.X509TrustManager;
 
 import de.danoeh.antennapod.core.preferences.UserPreferences;
+import de.danoeh.antennapod.core.service.UserAgentInterceptor;
 import de.danoeh.antennapod.core.storage.DBWriter;
 import okhttp3.Cache;
 import okhttp3.CipherSuite;
@@ -116,6 +117,7 @@ public class AntennapodHttpClient {
             return response;
         });
         builder.interceptors().add(new BasicAuthorizationInterceptor());
+        builder.networkInterceptors().add(new UserAgentInterceptor());
 
         // set cookie handler
         CookieManager cm = new CookieManager();
diff --git a/core/src/main/java/de/danoeh/antennapod/core/service/download/HttpDownloader.java b/core/src/main/java/de/danoeh/antennapod/core/service/download/HttpDownloader.java
index 46b8568c5..54b8d321a 100644
--- a/core/src/main/java/de/danoeh/antennapod/core/service/download/HttpDownloader.java
+++ b/core/src/main/java/de/danoeh/antennapod/core/service/download/HttpDownloader.java
@@ -63,8 +63,7 @@ public class HttpDownloader extends Downloader {
 
         try {
             final URI uri = URIUtil.getURIFromRequestUrl(request.getSource());
-            Request.Builder httpReq = new Request.Builder().url(uri.toURL())
-                    .header("User-Agent", ClientConfig.USER_AGENT);
+            Request.Builder httpReq = new Request.Builder().url(uri.toURL());
             httpReq.tag(request);
             if (request.getFeedfileType() == FeedMedia.FEEDFILETYPE_FEEDMEDIA) {
                 // set header explicitly so that okhttp doesn't do transparent gzip
-- 
cgit v1.2.3