Skip to content

fetchurl: only store 200 responses#944

Merged
zimbatm merged 1 commit intomasterfrom
fetchurl-ok
May 17, 2022
Merged

fetchurl: only store 200 responses#944
zimbatm merged 1 commit intomasterfrom
fetchurl-ok

Conversation

@zimbatm
Copy link
Copy Markdown
Member

@zimbatm zimbatm commented May 17, 2022

If I understand the docs properly, it should follow 30x redirects. We
don't want error responses, and things like 201 are also not interesting
to us.

Fixes #886

If I understand the docs properly, it should follow 30x redirects. We
don't want error responses, and things like 201 are also not interesting
to us.
@zimbatm zimbatm merged commit 6ac0fd6 into master May 17, 2022
@zimbatm zimbatm deleted the fetchurl-ok branch May 17, 2022 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

direnv fetchurl doesn't verify the HTTP status code

1 participant