From b539e28024eec3a5eda69efcb00fcc2acf7fe286 Mon Sep 17 00:00:00 2001 From: Josh Eckels Date: Wed, 24 Sep 2025 15:28:10 -0700 Subject: [PATCH] Backport: Timeout for querying for Uniprot features (#933) --- .../api-src/org/labkey/api/protein/fasta/FastaProtein.java | 2 +- protein/src/org/labkey/protein/ProteinServiceImpl.java | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/protein/api-src/org/labkey/api/protein/fasta/FastaProtein.java b/protein/api-src/org/labkey/api/protein/fasta/FastaProtein.java index aebfbd5291..039b7edf1e 100644 --- a/protein/api-src/org/labkey/api/protein/fasta/FastaProtein.java +++ b/protein/api-src/org/labkey/api/protein/fasta/FastaProtein.java @@ -84,7 +84,7 @@ public FastaProtein(String header, byte[] bytes) if (header.startsWith("gi|")) { firstAliasIndex = header.indexOf(" gi|", 2) + 1; - if (firstAliasIndex < 0 || firstAliasIndex > 30) + if (firstAliasIndex > 30) firstAliasIndex = 0; } diff --git a/protein/src/org/labkey/protein/ProteinServiceImpl.java b/protein/src/org/labkey/protein/ProteinServiceImpl.java index 269edcdb26..cae1ac8ea9 100644 --- a/protein/src/org/labkey/protein/ProteinServiceImpl.java +++ b/protein/src/org/labkey/protein/ProteinServiceImpl.java @@ -300,6 +300,8 @@ public List load(@NotNull String accession, @Nullable Object arg HttpURLConnection con = (HttpURLConnection) obj.openConnection(); con.setRequestProperty("Accept", "application/xml"); con.setRequestMethod("GET"); + con.setConnectTimeout(20_000); + con.setReadTimeout(20_000); int responseCode = con.getResponseCode(); if (responseCode == HttpURLConnection.HTTP_OK) { // success @@ -370,7 +372,7 @@ else if (location.getChildNodes().getLength() == 2) { if (responseCode != 404) { - LOG.error("HTTP GET failed to " + url + " with error code " + responseCode); + LOG.warn("HTTP GET failed to " + url + " with error code " + responseCode); } else {