~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/config.py

  • Committer: Vincent Ladeuil
  • Date: 2009-04-27 16:10:10 UTC
  • mto: (4310.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 4311.
  • Revision ID: v.ladeuil+lp@free.fr-20090427161010-7swfzeagf63cpixd
Fix bug #367726 by reverting some default user handling introduced
while fixing bug #256612.

* bzrlib/transport/ssh.py:
(_paramiko_auth): Explicitly use getpass.getuser() as default
user.

* bzrlib/transport/ftp/_gssapi.py:
(GSSAPIFtpTransport._create_connection): Explicitly use
getpass.getuser() as default user.

* bzrlib/transport/ftp/__init__.py:
(FtpTransport._create_connection): Explicitly use
getpass.getuser() as default user.

* bzrlib/tests/test_sftp_transport.py:
(TestUsesAuthConfig.test_sftp_is_none_if_no_config)
(TestUsesAuthConfig.test_sftp_doesnt_prompt_username): Revert to
None as the default user.

* bzrlib/tests/test_remote.py:
(TestRemoteSSHTransportAuthentication): The really offending one:
revert to None as the default user.

* bzrlib/tests/test_config.py:
(TestAuthenticationConfig.test_username_default_no_prompt): Update
test (and some PEP8).

* bzrlib/smtp_connection.py:
(SMTPConnection._authenticate): Revert to None as the default
user.

* bzrlib/plugins/launchpad/account.py:
(_get_auth_user): Revert default value handling.

* bzrlib/config.py:
(AuthenticationConfig.get_user): Fix doc-string. Leave default
value handling to callers.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1132
1132
        config.update({name: values})
1133
1133
        self._save()
1134
1134
 
1135
 
    def get_user(self, scheme, host, port=None, realm=None, path=None, 
 
1135
    def get_user(self, scheme, host, port=None, realm=None, path=None,
1136
1136
                 prompt=None, ask=False, default=None):
1137
1137
        """Get a user from authentication file.
1138
1138
 
1149
1149
        :param ask: Ask the user if there is no explicitly configured username 
1150
1150
                    (optional)
1151
1151
 
 
1152
        :param default: The username returned if none is defined (optional).
 
1153
 
1152
1154
        :return: The found user.
1153
1155
        """
1154
1156
        credentials = self.get_credentials(scheme, host, port, user=None,
1169
1171
                    prompt_host = host
1170
1172
                user = ui.ui_factory.get_username(prompt, host=prompt_host)
1171
1173
            else:
1172
 
                if default is None:
1173
 
                    import getpass
1174
 
                    default = getpass.getuser()
1175
1174
                user = default
1176
1175
        return user
1177
1176