Merge commit '8c13808' into master

Author: koppor

src/main/java/org/jabref/logic/importer/fetcher/DoiResolution.java

@@ -3,6 +3,7 @@
 import java.io.IOException;
 import java.net.URL;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.List;
 import java.util.Locale;
 import java.util.Objects;
@@ -31,6 +32,11 @@
 public class DoiResolution implements FulltextFetcher {
     private static final Logger LOGGER = LoggerFactory.getLogger(DoiResolution.class);
 
+    /**
+     * Hosts for which tailored fetchers exist, so this fetcher is not needed.
+     */
+    private final List<String> excludedHosts = Arrays.asList("link.springer.com", "ieeexplore.ieee.org");
+
     @Override
     public Optional<URL> findFullText(BibEntry entry) throws IOException {
         Objects.requireNonNull(entry);
@@ -57,7 +63,12 @@ public Optional<URL> findFullText(BibEntry entry) throws IOException {
             // some publishers are quite slow (default is 3s)
             connection.timeout(10000);
 
-            Document html = connection.get();
+            Connection.Response response = connection.execute();
+            if (excludedHosts.contains(response.url().getHost())) {
+                return Optional.empty();
+            }
+
+            Document html = response.parse();
             // scan for PDF
             Elements hrefElements = html.body().select("a[href]");
 

src/test/java/org/jabref/logic/importer/fetcher/DoiResolutionTest.java

@@ -53,6 +53,18 @@ void notReturnAnythingWhenMultipleLinksAreFound() throws IOException {
         assertEquals(Optional.empty(), finder.findFullText(entry));
     }
 
+    @Test
+    void notReturnAnythingWhenDOILeadsToSpringerLink() throws IOException {
+        entry.setField(StandardField.DOI, "https://doi.org/10.1007/978-3-319-89963-3_28");
+        assertEquals(Optional.empty(), finder.findFullText(entry));
+    }
+
+    @Test
+    void notReturnAnythingWhenDOILeadsToIEEE() throws IOException {
+        entry.setField(StandardField.DOI, "https://doi.org/10.1109/TTS.2020.2992669");
+        assertEquals(Optional.empty(), finder.findFullText(entry));
+    }
+
     @Test
     void notFoundByDOI() throws IOException {
         entry.setField(StandardField.DOI, "10.1186/unknown-doi");