GNU bug report logs - #26687
add git http support service

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 26687 in the body.
You can then email your comments to 26687 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Andy Wingo <wingo <at> igalia.com>
To: guix-patches <at> gnu.org
Subject: add git http support service
Date: Thu, 27 Apr 2017 22:17:14 +0200

[Message part 1 (text/plain, inline)]

Attached pair of patches adds git http "support" to the manual.  I say
"support" because really what you usually want to do is extend an
existing nginx server with git support, not make an entirely new
server.  But right now we can't extend servers, so this support is a bit
preliminary.  Still, it's better than ad hoc logic in peoples'
configuration files.

[0004-doc-Create-Version-Control-Services-section.patch (text/x-patch, attachment)]

[0005-gnu-Add-Git-HTTP-S-service-support.patch (text/x-patch, attachment)]

Message #8 received at 26687 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Andy Wingo <wingo <at> igalia.com>
Cc: 26687 <at> debbugs.gnu.org
Subject: Re: bug#26687: add git http support service
Date: Wed, 03 May 2017 14:11:39 +0200

Andy Wingo <wingo <at> igalia.com> skribis:

> Attached pair of patches adds git http "support" to the manual.  I say
> "support" because really what you usually want to do is extend an
> existing nginx server with git support, not make an entirely new
> server.  But right now we can't extend servers, so this support is a bit
> preliminary.  Still, it's better than ad hoc logic in peoples'
> configuration files.

Agreed, that’s a nice addition.

>>From 80d872ae8362b5e769c463599b03ddb0fb7482d2 Mon Sep 17 00:00:00 2001
> From: Andy Wingo <wingo <at> igalia.com>
> Date: Thu, 27 Apr 2017 10:59:44 +0200
> Subject: [PATCH 4/5] doc: Create Version Control Services section.
>
> * doc/guix.texi (Version Control Services): Move to its own node.

OK.

>>From cb9f33e2a70b304cc5712d8a369dc8db7a8ae95b Mon Sep 17 00:00:00 2001
> From: Andy Wingo <wingo <at> igalia.com>
> Date: Thu, 27 Apr 2017 15:02:19 +0200
> Subject: [PATCH 5/5] gnu: Add Git HTTP(S) service support.
>
> * doc/guix.texi (Version Control Services): Add documentation on the HTTP
> backend for git.
> * gnu/services/version-control.scm (<git-http-configuration>): New data type.
> (git-http-nginx-location-configuration): New helper function.

[...]

> +The @code{git://} protocol lacks authentication.  When you pull from a
> +repository fetched via @code{git://}, you don't know that the data you
> +receive was modified in transit or not.  It's better to use an

Nitpick: “modified in transit” refers to integrity, not authenticity;
perhaps write “you don’t know that the data you receive is really coming
from the specified host”?

Also maybe “lacks authentication and confidentiality”.

> +(define* (git-http-nginx-location-configuration #:optional
> +                                                (config
> +                                                 (git-http-configuration)))

Perhaps “git-http-configuration->nginx-location”?

OK with changes along these lines!

Eventually we should consider adding tests for the Git services.

Thank you,
Ludo’.

Message #13 received at 26687-done <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Andy Wingo <wingo <at> igalia.com>
Cc: 26687-done <at> debbugs.gnu.org
Subject: Re: bug#26687: add git http support service
Date: Tue, 28 Nov 2017 23:43:26 +0100

Hello!

I pushed those patches as commits
4313125816c0cfc9ce15ea1fec8a941976012ee6 and
514baaf8b50005cedf11e2c2cc3bce0e9d90328f.

I also added a test for the Git-over-HTTP service as commit
58ff6e1e0b1f1c4d1099aa317a0445d5ffcbd662.

Thanks!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.