# HG changeset patch # User Edouard Gomez # Date 1178923290 -7200 # Node ID 439e2f2fde42f184ed964712e5bc30b1f488883d # Parent fc8b3e7cbf6b26b4e5f20df14335710227564d54 Fix inconsistency for the stream_out capability in hgweb During some experiments of mine, the uncompressed cloning could not be enabled for hgweb.cgi nor hgwebdir.cgi though the server claimed to be stream_out capable. The only solution was to enable it using the user's .hgrc file. This solution is not acceptable when publishing the repos through an HTTP server because the CGI runs as a www dedicated user whose's home hgrc file may not be accessible to users publishing their repos through their userdir. For such cases we could end up with this typical debug output: hg --debug clone --uncompressed http://server/hg/project destination directory: project sending capabilities command capabilities: lookup changegroupsubset stream=1 unbundle=HG10GZ,HG10BZ,HG10UN sending stream_out command abort: operation forbidden by server The error lies in the fact the hgweb object defines new accessors to the repo configuration that trust things by default (untrusted=True) but the streamclone:stream_out function uses the usual accessors to the repo.ui object, which do not trust by default (untrusted=False) Fix this inconsistency, adding a new parameter to the stream_out function. hgweb then forces a "trust by default" behavior. diff --git a/mercurial/hgweb/hgweb_mod.py b/mercurial/hgweb/hgweb_mod.py --- a/mercurial/hgweb/hgweb_mod.py +++ b/mercurial/hgweb/hgweb_mod.py @@ -1177,4 +1177,4 @@ class hgweb(object): def do_stream_out(self, req): req.httphdr("application/mercurial-0.1") - streamclone.stream_out(self.repo, req) + streamclone.stream_out(self.repo, req, untrusted=True) diff --git a/mercurial/streamclone.py b/mercurial/streamclone.py --- a/mercurial/streamclone.py +++ b/mercurial/streamclone.py @@ -56,11 +56,11 @@ def walkrepo(root): # # server writes out raw file data. -def stream_out(repo, fileobj): +def stream_out(repo, fileobj, untrusted=False): '''stream out all metadata files in repository. writes to file-like object, must support write() and optional flush().''' - if not repo.ui.configbool('server', 'uncompressed'): + if not repo.ui.configbool('server', 'uncompressed', untrusted=untrusted): fileobj.write('1\n') return