[MERGE] Bug 84659 "serve --allow-writes" allows more than you might think

Danny van Heumen danny at dannyvanheumen.nl
Thu Jul 23 19:14:23 BST 2009


Hi,

Bug: https://bugs.launchpad.net/bzr/+bug/84659 (Easy)

This bug is about the '--allow-writes' switch that also opens up the
server for anonymous writes, which is not immediately clear from the
name of the switch.

I've fixed this issue by adding a warning. In the case where
'--allow-writes' is called and '--inet' isn't, this warning will be
displayed.
The blackbox tests are fixed accordingly. (I'm assuming that we don't
need a separate test for testing just the warning message.)
(I've incorporated all the feedback Martin Pool gave me.)

Branch:
lp:~danny.vanheumen/bzr/84659-serve--allow-writes-allows-more-than-you-might-think


I hope that I'm doing everything right, since this is my very first code
contribution to bazaar (and using launchpad). So, please just let me
know if I do something wrong.

Kind regards,

Danny



More information about the bazaar mailing list