Skip to content

Commit 7abf58e

Browse files
authored
Merge pull request #382 from Joragasy/fix-handle-head-request-fallback-to-get
fix: fallback to GET when HEAD request returns non-2xx response
2 parents afc3b60 + ef7e4f7 commit 7abf58e

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

ckanext/dcat/harvesters/base.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,9 @@ def _get_content_and_type(self, url, harvest_job, page=1,
6767
did_get = False
6868
r = session.head(url)
6969

70-
if r.status_code == 405 or r.status_code == 400:
70+
# Some servers respond with 400 or 404 to HEAD requests, even if the resource exists.
71+
# In that case we want to try a GET request before giving up.
72+
if not r.ok:
7173
r = session.get(url, stream=True)
7274
did_get = True
7375
r.raise_for_status()

0 commit comments

Comments
 (0)