git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] git-remote-mediawiki: better error message when HTTP(S) access fails

From
Jeff King <peff@peff.net>
Date
May 29, 2013, 15:26 UTC
Message-ID
<20130529152653.GB9072@sigill.intra.peff.net>
In-Reply-To
<1369829189-27881-1-git-send-email-Matthieu.Moy@imag.fr>
On Wed, May 29, 2013 at 02:06:29PM +0200, Matthieu Moy wrote:
Show 7 quoted lines
> My use-case is an invalid SSL certificate. Pulling from the wiki with a
> recent version of libwww-perl fails, and git-remote-mediawiki gave no
> clue about the reason. Give the mediawiki API detailed error message, and
> since it is not so informative, hint the user about an invalid SSL
> certificate on https:// urls.
> 
> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
Thanks, this looks like a reasonable stopping point for now.

There are still some calls that do not check the result of the mediawiki calls at all, and I think they'll probably cause a perl error when they fail (for dereferencing undef). But these ones are the initial contact, so they should be the common ones to fail. Refactoring the whole error handling is a bit larger task and can wait until somebody is interested.

-Peff
Previous: Matthieu MoyNext: Jeff King
Message 5 of 6 in “git-remote-mediawiki: better error message when HTTP(S) access fails”
  1. git-remote-mediawiki: better error message when HTTP(S) access failsMatthieu Moy, May 23, 2013
  2. Jeff KingMay 28, 2013
  3. Matthieu MoyMay 29, 2013
  4. git-remote-mediawiki: better error message when HTTP(S) access failsMatthieu Moy, May 29, 2013
  5. Jeff KingMay 29, 2013
  6. Jeff KingMay 29, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.