Merge pull request #399 from arXiv/fix-expires-header

Fix expires header
arXiv · Oct 9, 2023 · d40bf9a · d40bf9a
2 parents 9f6ab59 + 1ea91ea
commit d40bf9a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/browse/controllers/archive_page/__init__.py b/browse/controllers/archive_page/__init__.py
@@ -108,7 +108,7 @@ def category_list(archive_id: str) -> List[Dict[str, str]]:
 
 def _write_expires_header(response_headers: Dict[str, Any]) -> None:
     """Writes an expires header for the response."""
-    response_headers["Expires"] = abs_expires_header(biz_tz())[1]
+    response_headers["Expires"] = abs_expires_header(biz_tz())
 
 
 DAYS = ["{:0>2d}".format(i) for i in range(1, 32)]

diff --git a/browse/controllers/response_headers.py b/browse/controllers/response_headers.py
@@ -67,13 +67,13 @@ def guess_next_update_utc(arxiv_business_tz: ZoneInfo, dt: Optional[datetime] =
     return (possible_publish_dt.astimezone(tz=timezone.utc), likely_in_publish)
 
 
-def abs_expires_header(arxiv_business_tz: ZoneInfo) -> Tuple[str, str]:
+def abs_expires_header(arxiv_business_tz: ZoneInfo) -> str:
     """Get the expires header key and value that should be used by abs."""
     (next_update_dt, likely_in_publish) = guess_next_update_utc(arxiv_business_tz)
     if likely_in_publish:
-        return ('Expires', '-1')
+        return '-1'
     else:
-        return ('Expires', mime_header_date(next_update_dt))
+        return mime_header_date(next_update_dt)
 
 
 def mime_header_date(dt: datetime) -> str:

diff --git a/tests/test_browse.py b/tests/test_browse.py
@@ -375,10 +375,10 @@ def test_long_author_colab(self):
 
         self.assertIsNotNone(
             colab['href'], '<a> tag in title should have href')
-        self.assertEqual(
-            colab['href'], 'https://arxiv.org/search/physics?searchtype=author&query=ILL%2FESS%2FLiU+collaboration')
-        self.assertEqual(
-            colab.text, 'ILL/ESS/LiU collaboration for the development of the B10 detector technology in the framework of the CRISP project')
+        self.assertRegex(colab['href'],
+                         'search\/physics\?searchtype=author\&query=ILL.ESS.LiU\+collaboration')
+        self.assertEqual(colab.text,
+                         'ILL/ESS/LiU collaboration for the development of the B10 detector technology in the framework of the CRISP project')
 
     @unittest.skip("In current implementation, conflicts with comma test below.")
     def test_space_in_author_list(self):