# HG changeset patch
# User Mark Edgington <edgimar@gmail.com>
# Date 1231012206 -3600
# Node ID bbcd2dea19fef832dbcb4c7a4f0b085d1ee50c2a
# Parent  b663b5563de701ff2740d1400c27d262087678d7
hgweb: send HTTP unauthorized error when denying pull

diff -r b663b5563de7 -r bbcd2dea19fe mercurial/hgweb/hgweb_mod.py
--- a/mercurial/hgweb/hgweb_mod.py	Sat Jan 03 20:33:19 2009 +0100
+++ b/mercurial/hgweb/hgweb_mod.py	Sat Jan 03 20:50:06 2009 +0100
@@ -289,7 +289,7 @@
             raise ErrorResponse(HTTP_UNAUTHORIZED, 'read not authorized')
 
         if op == 'pull' and not self.allowpull:
-            raise ErrorResponse(HTTP_OK, '')
+            raise ErrorResponse(HTTP_UNAUTHORIZED, 'pull not authorized')
         # op is None when checking allow/deny_read permissions for a web-browser request
         elif op == 'pull' or op is None:
             return
diff -r b663b5563de7 -r bbcd2dea19fe tests/test-pull-http.out
--- a/tests/test-pull-http.out	Sat Jan 03 20:33:19 2009 +0100
+++ b/tests/test-pull-http.out	Sat Jan 03 20:50:06 2009 +0100
@@ -2,11 +2,11 @@
 updating working directory
 1 files updated, 0 files merged, 0 files removed, 0 files unresolved
 % expect error, cloning not allowed
-abort: error: 
+abort: authorization failed
 requesting all changes
 % serve errors
 % expect error, pulling not allowed
-abort: error: 
+abort: authorization failed
 pulling from http://localhost/
 searching for changes
 % serve errors