[bug#53060] [PATCH 0/2] Allow the github updater to update git sources

[Ludovic Courtès]

Hi,

Maxime Devos <maximedevos@telenet.be> skribis:

> tests/cpan.scm uses 'with-http-server', which I do not find ideal
> because the answers the HTTP server gives depend on the order the
> HTTP server was queried, without verifying the URI.  Mocking
> 'http-fetch' allows me not to worry about ordering and allows verifying
> the URI.

Good point.

> It might be possible to modify 'with-http-server' into something
> (with-http-server*?) that allows looking at the HTTP headers and URI
> and dynamically generate a response based on that.

Yes, that’d be great.

> Due to the mocking, %github-api isn't truly necessary, but having
> "https://api.github.com"; in a single location helps avoiding typos
> like writing "http://"; instead of "https://"; somewhere, or adjusting
> the domain name if GitHub decided to change it for whatever reason
> (hopefully unlikely?), or if Tor becomes very popular among the general
> public and GitHub has an ".onion" address, then it could be changed to
> the ".onion" address easily, ...

Yes, and also, setting ‘%github-api’ makes sure we don’t inadvertently
talk to the real GitHub (there was a bug along these lines in the tests
of one of the importers a while back.)

Thanks,
Ludo’.