[MERGE][1.2][#185394] Disconnect and reconnect the smart medium after getting "unknown method" error from server if a request body was sent.
Martin Pool
mbp at canonical.com
Thu Feb 7 00:42:26 GMT 2008
Martin Pool has voted tweak.
Status is now: Conditionally approved
Comment:
My biggest concern is the disruption to users who enter their password
on
every reconnection. So I think we should at least give them a message
explaining what is happening and why. Something like:
Server is too old for streaming push, reconnecting...
And I would say the NEWS entry should be right at the top in the "notes
when upgrading", and explain that this might require you to reenter a
password or passphrase.
If this is 1.2 it will make deployment of 1.2 onto Launchpad urgent; it
really should be there before 1.2final.
+ # Be optimistic: we assume the remote end can accept new remote
+ # requests until we get an error saying otherwise. (1.2 adds
some
+ # requests that send bodies, which confuses older servers.)
+ self._remote_is_at_least_1_2 = True
It might be more accurate as a ternary None/True/False, but it's
probably
not worth it.
The fact that we have to do this here indicates to me the value of
having
a machine-parseable indicator of the version in messages, even though
we'll generally try to avoid making decisions based on it. Not to
mention having the body explicitly marked. So we should try to get on
with doing the protocol 3 changes, and make sure that they consider how
we detect protocol version.
For details, see:
http://bundlebuggy.aaronbentley.com/request/%3C20080206060316.GB5911%40steerpike.home.puzzling.org%3E
More information about the bazaar
mailing list