GNU bug report logs -
#36433
gnus-read-ephemeral-bug-group does not handle errors, leaving temporary files behind
Previous Next
Reported by: Tim Landscheidt <tim <at> tim-landscheidt.de>
Date: Sat, 29 Jun 2019 12:58:01 UTC
Severity: normal
Tags: fixed, patch
Found in version 5.13
Fixed in version 27.1
Done: Lars Ingebrigtsen <larsi <at> gnus.org>
Bug is archived. No further changes may be made.
Full log
Message #8 received at 36433 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
tags 36433 + patch
quit
[0001-Improve-Gnus-ephemeral-bug-group-browsing.patch (text/x-diff, attachment)]
[Message part 3 (text/plain, inline)]
Tim Landscheidt <tim <at> tim-landscheidt.de> writes:
> With Emacs 26.1, M-x gnus-read-ephemeral-bug-group RET
> foobar RET gives "gnus-list-of-unread-articles: Group
> nndoc+ephemeral:bug#0 couldn’t be activated", while leaving
> a temporary file with the contents:
>
> | <HTML>
> | <HEAD><TITLE>Error</TITLE></HEAD>
> | <BODY>
> | An error occurred.
> | Error was: No bug number
> | </BODY></HTML>
>
> behind in /tmp.
>
> This also happens with gnus-read-ephemeral-emacs-bug-group
> and gnus-read-ephemeral-debian-bug-group.
The attached patch fixes this for Emacs 27. It also improves the
reading and handling of multiple string/numeric bug IDs, which is
currently quite brittle, and extends some of the relevant documentation.
WDYT?
> For GNU bugs, fixing this might be made easier by updating
> debbugs.gnu.org as it currently returns 200 OK for non-ex-
> isting bugs:
>
> | [tim <at> passepartout ~]$ curl -is 'https://debbugs.gnu.org/cgi/bugreport.cgi?bug=foobar;mboxmaint=yes;mboxstat=yes' | head -1
> | HTTP/1.1 200 OK
> | [tim <at> passepartout ~]$
>
> while bugs.debian.org gives a status code that can be used
> for diagnostics:
>
> | [tim <at> passepartout ~]$ curl -is 'https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=595256;mboxmaint=yes;mboxstat=yes' | head -1
> | HTTP/1.1 200 OK
> | [tim <at> passepartout ~]$ curl -is 'https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=4711;mboxmaint=yes;mboxstat=yes' | head -1
> | HTTP/1.1 404 Not Found
> | [tim <at> passepartout ~]$
I think the attached patch is as good as gnus-read-ephemeral-bug-group
can do for now, without fixing several known deficiencies of the url.el
library and debbugs.gnu.org instance (each of which deserves its own bug
ticket if it doesn't already have one).
Thanks,
--
Basil
This bug report was last modified 5 years and 318 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.