Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resource considered as valid whereas the distant resource is 404 #1048

Closed
thom4parisot opened this issue Jul 24, 2017 · 4 comments
Closed

Resource considered as valid whereas the distant resource is 404 #1048

thom4parisot opened this issue Jul 24, 2017 · 4 comments
Labels

Comments

@thom4parisot
Copy link
Contributor

thom4parisot commented Jul 24, 2017

Detailed Description

Cf. https://www.data.gouv.fr/fr/datasets/allocataires-du-rsa-30378531/

image

$ curl --head http://drees.solidarites-sante.gouv.fr/IMG/xls/rsa_donnees_departementales.xls
HTTP/1.1 404 Not Found
Date: Mon, 24 Jul 2017 11:11:58 GMT
Content-Type: text/html; charset=iso-8859-1
Cache-Control: max-age=3600
Vary: Accept-Encoding
Connection: close

Context

When a resource is displayed as existing , I want to assume it is true , so I can count on the resource to work with.

Possible Implementation

N/A

Your Environment

  • Browser Name and version: Chrome Evergreen
  • Operating System and version (desktop or mobile): macOS 10.12.6 (16G29)
@davidbgk
Copy link
Member

It may be on the croquemort side. /cc @abulte

@abulte
Copy link
Contributor

abulte commented Sep 5, 2017

There is a problem indeed on the udata side, the croquemort response does identify a 404: {"charset": "iso-8859-1", "content-disposition": "", "content-encoding": "gzip", "content-length": "198", "content-location": "", "content-md5": "", "content-type": "text/html", "etag": "", "expires": "", "group": "bf664fdb", "last-modified": "", "status": "404", "updated": "2017-09-05T10:40:26.551548", "url": "http://www.drees.sante.gouv.fr/IMG/xls/rsa_donnees_departementales.xls"}.

The integration is being overhauled though, this will be fixed.

abulte added a commit that referenced this issue Sep 5, 2017
@thom4parisot
Copy link
Contributor Author

Cool, to be checked on https://next.data.gouv.fr/fr/datasets/liste-des-prenoms-2010-de-digne-les-bains-op/ when (re)deployed.

@thom4parisot
Copy link
Contributor Author

👍

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants