Lock keyAndVersionExtracted while being read/written.

Author: FireMasterK

extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeParsingHelper.java

@@ -201,6 +201,7 @@ private YoutubeParsingHelper() {
     private static String[] youtubeMusicKey;
 
     private static boolean keyAndVersionExtracted = false;
+    private static final Object KEY_AND_VERSION_EXTRACTED_LOCK = new Object();
     @SuppressWarnings("OptionalUsedAsFieldOrParameterType")
     private static Optional<Boolean> hardcodedClientVersionAndKeyValid = Optional.empty();
 
@@ -619,82 +620,89 @@ public static boolean areHardcodedClientVersionAndKeyValid()
 
     private static void extractClientVersionAndKeyFromSwJs()
             throws IOException, ExtractionException {
-        if (keyAndVersionExtracted) {
-            return;
-        }
-        final String url = "https://www.youtube.com/sw.js";
-        final Map<String, List<String>> headers = new HashMap<>();
-        headers.put("Origin", singletonList("https://www.youtube.com"));
-        headers.put("Referer", singletonList("https://www.youtube.com"));
-        final String response = getDownloader().get(url, headers).responseBody();
-        try {
-            clientVersion = getStringResultFromRegexArray(response,
-                    INNERTUBE_CONTEXT_CLIENT_VERSION_REGEXES, 1);
-            key = getStringResultFromRegexArray(response, INNERTUBE_API_KEY_REGEXES, 1);
-        } catch (final Parser.RegexException e) {
-            throw new ParsingException("Could not extract YouTube WEB InnerTube client version "
-                    + "and API key from sw.js", e);
+        synchronized (KEY_AND_VERSION_EXTRACTED_LOCK) {
+            if (keyAndVersionExtracted) {
+                return;
+            }
+            final String url = "https://www.youtube.com/sw.js";
+            final Map<String, List<String>> headers = new HashMap<>();
+            headers.put("Origin", singletonList("https://www.youtube.com"));
+            headers.put("Referer", singletonList("https://www.youtube.com"));
+            final String response = getDownloader().get(url, headers).responseBody();
+            try {
+                clientVersion = getStringResultFromRegexArray(response,
+                        INNERTUBE_CONTEXT_CLIENT_VERSION_REGEXES, 1);
+                key = getStringResultFromRegexArray(response, INNERTUBE_API_KEY_REGEXES, 1);
+            } catch (final Parser.RegexException e) {
+                throw new ParsingException("Could not extract YouTube WEB InnerTube client version "
+                        + "and API key from sw.js", e);
+            }
+            keyAndVersionExtracted = true;
         }
-        keyAndVersionExtracted = true;
     }
 
     private static void extractClientVersionAndKeyFromHtmlSearchResultsPage()
             throws IOException, ExtractionException {
-        // Don't extract the client version and the InnerTube key if it has been already extracted
-        if (keyAndVersionExtracted) {
-            return;
-        }
 
-        // Don't provide a search term in order to have a smaller response
-        final String url = "https://www.youtube.com/results?search_query=&ucbcb=1";
-        final String html = getDownloader().get(url, getCookieHeader()).responseBody();
-        final JsonObject initialData = getInitialData(html);
-        final JsonArray serviceTrackingParams = initialData.getObject("responseContext")
-                .getArray("serviceTrackingParams");
+        synchronized (KEY_AND_VERSION_EXTRACTED_LOCK) {
 
-        // Try to get version from initial data first
-        final Stream<JsonObject> serviceTrackingParamsStream = serviceTrackingParams.stream()
-                .filter(JsonObject.class::isInstance)
-                .map(JsonObject.class::cast);
+            // Don't extract the client version and the InnerTube key
+            // if it has been already extracted
+            if (keyAndVersionExtracted) {
+                return;
+            }
 
-        clientVersion = getClientVersionFromServiceTrackingParam(
-                serviceTrackingParamsStream, "CSI", "cver");
+            // Don't provide a search term in order to have a smaller response
+            final String url = "https://www.youtube.com/results?search_query=&ucbcb=1";
+            final String html = getDownloader().get(url, getCookieHeader()).responseBody();
+            final JsonObject initialData = getInitialData(html);
+            final JsonArray serviceTrackingParams = initialData.getObject("responseContext")
+                    .getArray("serviceTrackingParams");
+
+            // Try to get version from initial data first
+            final Stream<JsonObject> serviceTrackingParamsStream = serviceTrackingParams.stream()
+                    .filter(JsonObject.class::isInstance)
+                    .map(JsonObject.class::cast);
+
+            clientVersion = getClientVersionFromServiceTrackingParam(
+                    serviceTrackingParamsStream, "CSI", "cver");
+
+            if (clientVersion == null) {
+                try {
+                    clientVersion = getStringResultFromRegexArray(html,
+                            INNERTUBE_CONTEXT_CLIENT_VERSION_REGEXES, 1);
+                } catch (final Parser.RegexException ignored) {
+                }
+            }
+
+            // Fallback to get a shortened client version which does not contain the last two
+            // digits
+            if (isNullOrEmpty(clientVersion)) {
+                clientVersion = getClientVersionFromServiceTrackingParam(
+                        serviceTrackingParamsStream, "ECATCHER", "client.version");
+            }
 
-        if (clientVersion == null) {
             try {
-                clientVersion = getStringResultFromRegexArray(html,
-                        INNERTUBE_CONTEXT_CLIENT_VERSION_REGEXES, 1);
+                key = getStringResultFromRegexArray(html, INNERTUBE_API_KEY_REGEXES, 1);
             } catch (final Parser.RegexException ignored) {
             }
-        }
 
-        // Fallback to get a shortened client version which does not contain the last two
-        // digits
-        if (isNullOrEmpty(clientVersion)) {
-            clientVersion = getClientVersionFromServiceTrackingParam(
-                    serviceTrackingParamsStream, "ECATCHER", "client.version");
-        }
-
-        try {
-            key = getStringResultFromRegexArray(html, INNERTUBE_API_KEY_REGEXES, 1);
-        } catch (final Parser.RegexException ignored) {
-        }
+            if (isNullOrEmpty(key)) {
+                throw new ParsingException(
+                        // CHECKSTYLE:OFF
+                        "Could not extract YouTube WEB InnerTube API key from HTML search results page");
+                // CHECKSTYLE:ON
+            }
 
-        if (isNullOrEmpty(key)) {
-            throw new ParsingException(
-                    // CHECKSTYLE:OFF
-                    "Could not extract YouTube WEB InnerTube API key from HTML search results page");
-                    // CHECKSTYLE:ON
-        }
+            if (clientVersion == null) {
+                throw new ParsingException(
+                        // CHECKSTYLE:OFF
+                        "Could not extract YouTube WEB InnerTube client version from HTML search results page");
+                // CHECKSTYLE:ON
+            }
 
-        if (clientVersion == null) {
-            throw new ParsingException(
-                    // CHECKSTYLE:OFF
-                    "Could not extract YouTube WEB InnerTube client version from HTML search results page");
-                    // CHECKSTYLE:ON
+            keyAndVersionExtracted = true;
         }
-
-        keyAndVersionExtracted = true;
     }
 
     @Nullable