aboutsummaryrefslogtreecommitdiff
path: root/doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn
diff options
context:
space:
mode:
Diffstat (limited to 'doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn')
-rw-r--r--doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn8
1 files changed, 7 insertions, 1 deletions
diff --git a/doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn b/doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn
index 5015b6290..db49f1e4a 100644
--- a/doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn
+++ b/doc/bugs/Http_error_500_when_using_mercurial_backend.mdwn
@@ -19,4 +19,10 @@ Here is a quick patch, partialy copying the svn backend.
+ $message="web commit by Anonymous".(length $message ? ": $message" : "");
}
- $message = possibly_foolish_untaint($message); \ No newline at end of file
+ $message = possibly_foolish_untaint($message);
+
+> The svn backend puts the user info in the message because that's the only
+> way to store the user info, unlike with mercurial. The svn plugin also
+> removes that info when getting the RecentChanges info. Since mercurial
+> does not do that, it seemed better to me to test for an empty message and
+> set it to a dummy commit string, which I've [[done]]. --[[Joey]]