[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26489: [PATCH] substitute: Ignore bad responses.
From: |
Ludovic Courtès |
Subject: |
bug#26489: [PATCH] substitute: Ignore bad responses. |
Date: |
Fri, 14 Apr 2017 11:54:32 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Howdy!
Tobias Geerinckx-Rice <address@hidden> skribis:
> * guix/scripts/substitute.scm (http-multiple-get): Catch BAD-RESPONSE
> exceptions and keep going.
> ---
>
> Guix,
>
> One weird HTTP response from a server will kill ‘guix substitute’:
>
> updating list of substitutes from 'https://foo'... 50.0%Backtrace:
> ...
> guix/ui.scm:1229:8: In procedure run-guix-command:
> guix/ui.scm:1229:8: Throw to key `bad-response' with args
> `("Bad Response-Line: ~s" (""))'.
> error: build failed: substituter `substitute' died unexpectedly
>
> Attached is a patch to ignore such bad responses. The offending .narinfo
> will be ignored for that session, and not cached at all. The result:
I’m sure you expect this question: what bad responses did you get in
practice? :-)
Usually that is a sign of a broken HTTP server. Of course it’s
widespread enough, we’d better handle it, either in Guix or directly in
(web client) in Guile; OTOH, if it’s a genuine problem, we’d better not
hide it.
Thanks,
Ludo’.