diff options
| author | Jeff Forcier <jeff@bitprophet.org> | 2023-05-24 15:52:33 -0400 |
|---|---|---|
| committer | Jeff Forcier <jeff@bitprophet.org> | 2023-05-24 15:52:43 -0400 |
| commit | 098eea5d97a9985d7ab965d8b7ab8425781ebb5e (patch) | |
| tree | bdb88bfabc6c1ad4eefc8c864e5003105db2952f | |
| parent | fdb08b7cb94d9edb547790a20ca28cfd45d20c53 (diff) | |
AuthStrategy get_sources doesn't need transport arg anymore
| -rw-r--r-- | paramiko/auth_strategy.py | 2 | ||||
| -rw-r--r-- | tests/auth.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/paramiko/auth_strategy.py b/paramiko/auth_strategy.py index 3da6d423..03c1d877 100644 --- a/paramiko/auth_strategy.py +++ b/paramiko/auth_strategy.py @@ -271,7 +271,7 @@ class AuthStrategy: # wants to live in fabric.OpenSSHAuthStrategy as not all target servers # will implement it! # TODO: needs better "server told us too many attempts" checking! - for source in self.get_sources(transport): + for source in self.get_sources(): self.log.debug(f"Trying {source}") try: # NOTE: this really wants to _only_ wrap the authenticate()! result = source.authenticate(transport) diff --git a/tests/auth.py b/tests/auth.py index a74c1f2c..52544e88 100644 --- a/tests/auth.py +++ b/tests/auth.py @@ -537,7 +537,7 @@ class AuthStrategy_: with raises(TypeError): self.strat.authenticate() result = self.strat.authenticate(self.transport) - self.strat.get_sources.assert_called_once_with(self.transport) + self.strat.get_sources.assert_called_once_with() self.source.authenticate.assert_called_once_with(self.transport) assert isinstance(result, AuthResult) assert result.strategy is self.strat |