Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Migrate rest of existing agent tests to new module387-auth-mark-ii | Jeff Forcier | 2023-05-17 | 2 | -6/+72 |
| | | | | | | Also make them better. Also fix AttributeError raising to not be py3.10 specific | ||||
* | Test more of recent AgentKey additions | Jeff Forcier | 2023-05-16 | 2 | -65/+25 |
| | |||||
* | Implement _fields on AgentKey so __eq__ works correctly | Jeff Forcier | 2023-05-16 | 3 | -1/+28 |
| | |||||
* | Sick of the warnings about this and don't use it myself | Jeff Forcier | 2023-05-16 | 1 | -2/+0 |
| | |||||
* | Use new Invocations combo check task | Jeff Forcier | 2023-05-16 | 2 | -3/+4 |
| | |||||
* | Partial implementation of new AuthStrategy mechanism re #387 | Jeff Forcier | 2023-05-09 | 4 | -1/+277 |
| | |||||
* | Made PKey.from_path cert-aware | Jeff Forcier | 2023-05-09 | 7 | -10/+127 |
| | | | | | | | This was previously only done in SSHClient. It's not relevant for from_type_string which is aimed at ssh-agents, which tend to do their own cert loading where necessary | ||||
* | Migrate cert related tests to newer pkey module | Jeff Forcier | 2023-05-08 | 12 | -104/+59 |
| | | | | | | - Merge them but also break them up. It's complicated. - Move cert files into _support - Related comments in the source as some of this is non-intuitive | ||||
* | Twiddle RSA signature algorithm fallback behavior in AuthOnlyHandler | Jeff Forcier | 2023-05-05 | 3 | -135/+189 |
| | | | | Also includes more rearranging of AuthHandler related tests | ||||
* | Move auth tests to be new style filename, obj naming | Jeff Forcier | 2023-05-05 | 2 | -12/+15 |
| | | | | Also allow test task module selector to see new-style test modules | ||||
* | Modernize auth tests to use shared server manager | Jeff Forcier | 2023-05-05 | 2 | -183/+65 |
| | |||||
* | Start consolidating test server nonsense | Jeff Forcier | 2023-05-05 | 2 | -179/+264 |
| | |||||
* | Start using git-blame-ignore-revs | Jeff Forcier | 2023-05-05 | 1 | -0/+2 |
| | | | | | | This file location/name is a GitHub-honored convention, and I've set up my local clone with `git config blame.ignoreRevsFile .git-blame-ignore-revs` to match. | ||||
* | Allow transport tests' server to specify which class to use | Jeff Forcier | 2023-05-05 | 1 | -2/+7 |
| | |||||
* | Refactor pubkey algo fallback | Jeff Forcier | 2023-05-05 | 1 | -7/+15 |
| | |||||
* | Test proving the issue driving most use of disabled_algorithms | Jeff Forcier | 2023-05-05 | 1 | -0/+22 |
| | | | | | Phrased in a it-passes-now fashion, but once fixed in main code path, will need changing of course. | ||||
* | Comment updates | Jeff Forcier | 2023-05-05 | 2 | -1/+3 |
| | |||||
* | Think we need pytest.ini in the sdist now if its tests are to work when ↵ | Jeff Forcier | 2023-05-05 | 1 | -1/+1 |
| | | | | distributed | ||||
* | Enhance PKey.from_path and test it better | Jeff Forcier | 2023-05-05 | 8 | -33/+58 |
| | |||||
* | Migrate rest of main keys and update suite to be more pytest-relaxed compat | Jeff Forcier | 2023-05-05 | 22 | -78/+199 |
| | | | | | | | | | | | | | | Main branch as of today: 350 passed, 21 skipped, 52 deselected, 3 warnings in 11.10s This branch as of this commit: 361 passed, 21 skipped, 52 deselected, 3 warnings in 10.51s Of those 11 "new" tests, 8 are ones I wrote (tests/pkey.py). Hard to figure out what the other 3 are given pytest-relaxed's output is very different from regular verbose pytest. oops. | ||||
* | Start moving key fixtures into support folder | Jeff Forcier | 2023-05-05 | 6 | -9/+13 |
| | |||||
* | Opt-in overhaul to how MSG_SERVICE_REQUEST is done | Jeff Forcier | 2023-05-05 | 5 | -34/+376 |
| | | | | | | | | | | | | | | | | | | - New subclass(es) for opt-in use. Most below messages refer to them, not parent classes. - In parent classes, make handler tables instance attributes for easier subclass twiddling. - Refactor Transport-level session check - Refactor Transport-level auth handler instantiation (but keep behavior the same, for now) - Add service-request handler to Transport subclass, and remove from AuthHandler subclass - Remove manual event injection from the handful of Transport auth methods which supported it. Suspect unused, don't need the extra complexity, and wasn't consistent anyways - can add back smarter later if anyone needs it. - Not bothering with gssapi at all for now as I cannot easily test it - Primarily tested against the new AuthStrategy architecture | ||||
* | Enhance AgentKey with comment, inner_key attributes | Jeff Forcier | 2023-05-05 | 3 | -8/+72 |
| | | | | | | | | | | | - Comment was being read-but-not-stored from the agent reply. wat? - Use newly added PKey constructor to instantiate a key subclass for the 'inner'/proxied key, this way client code can obtain stuff like bit size, fingerprint, etc. - Proxy to inner_key with __getattr__ so clients don't have to know whether they're dealing with an AgentKey or a regular one - Add `__repr__` to PKey instead of doing it in AgentKey. (wow, how did we not have this ever?) | ||||
* | Enhance PKey a bunch | Jeff Forcier | 2023-05-05 | 9 | -52/+153 |
| | | | | | | | | | - add .name to eventually replace .get_name - use that in a bunch of spots to avoid some duplication - add .identifiers classmethod to extend existing idea from ECDSAKey - add from_type_string alt constructor which uses .identifiers - use that in HostKeys (includes hopefully-minor refactoring) - no longer giving outdated init kwarg to ECDSA host key loading | ||||
* | Remove outdated/bad test fixture | Jeff Forcier | 2023-05-05 | 1 | -3/+0 |
| | | | | | | | | | | | This was added for #2173 but seemingly in its initial pass, before the submitter realized two spaces is _not_ a valid separator. It accidentally passed the tests due to how HostKeys.from_line was implemented at the time (two spaces -> 'key type' was misread and an empty string -> empty string not a valid key type -> returns None). An impending rewrite of that method turned this up. | ||||
* | Fix inaccurate PKey.__init__ docstring: data is bytes, not str | Jeff Forcier | 2023-05-05 | 1 | -2/+2 |
| | | | | | This seems likely a missed spot from the Python 3 conversion or the Python 2 drop. | ||||
* | Add ed25519 and ecdsa key types to hostkeys test fixtures | Jeff Forcier | 2023-05-05 | 1 | -7/+9 |
| | |||||
* | Add algorithm_name property to PKey | Jeff Forcier | 2023-05-05 | 3 | -0/+24 |
| | |||||
* | Add new PKey.fingerprint property at long last | Jeff Forcier | 2023-05-05 | 3 | -1/+31 |
| | |||||
* | Fix now-broken-on-py3 AgentKey.__str__, by deleting it | Jeff Forcier | 2023-05-05 | 3 | -3/+10 |
| | |||||
* | Random comment adds | Jeff Forcier | 2023-05-05 | 2 | -0/+7 |
| | |||||
* | Tinker a bit with tasks re: doc watching, doc browsing | Jeff Forcier | 2023-04-16 | 2 | -2/+5 |
| | | | | Requires an inflight set of changes to invocations | ||||
* | Version bump for dev | Jeff Forcier | 2023-04-12 | 1 | -1/+1 |
| | |||||
* | Merge branch '3.0' into 3.1HEADmain3.1 | Jeff Forcier | 2023-04-12 | 3 | -7/+3 |
|\ | |||||
| * | Use newer circleci orb to stop using codecov's old pypi package3.0 | Jeff Forcier | 2023-04-12 | 3 | -7/+3 |
| | | | | | | | | Closes #2226 | ||||
* | | Cut 3.1.03.1.0 | Jeff Forcier | 2023-03-10 | 2 | -1/+2 |
| | | |||||
* | | codespell | Jeff Forcier | 2023-03-10 | 1 | -1/+1 |
| | | |||||
* | | Changelog re #2013, closes #2009 | Jeff Forcier | 2023-03-10 | 1 | -0/+5 |
| | | |||||
* | | Merge branch 'main' into 2013-int | Jeff Forcier | 2023-03-10 | 22 | -59/+138 |
|\ \ | |||||
| * | | flake8 re #2193 | Jeff Forcier | 2023-03-03 | 1 | -2/+2 |
| | | | |||||
| * | | Merge pull request #2193 from AstraLuma/server-agent-docs | Jeff Forcier | 2023-03-03 | 2 | -0/+17 |
| |\ \ | | | | | | | | | Expand documentation on server agent handling. | ||||
| | * | | Expand documentation on server agent handling. | Jamie Bliss | 2023-02-24 | 2 | -0/+17 |
| | | | | |||||
| * | | | Use releases 2.1 to hide 1.x, 2.x unreleased bits in changelog | Jeff Forcier | 2023-02-24 | 2 | -1/+4 |
| | | | | |||||
| * | | | s/os.path/pathlib/ in www sphinx conf | Jeff Forcier | 2023-02-24 | 1 | -5/+6 |
| |/ / | |||||
| * | | Add credit in changelog re #2173 | Jeff Forcier | 2023-02-16 | 1 | -1/+1 |
| | | | |||||
| * | | spacing to pass style check | Alex Chavkin | 2023-02-16 | 1 | -0/+2 |
| | | | |||||
| * | | Update tests for #2173 | Alex Chavkin | 2023-02-16 | 1 | -1/+17 |
| | | | |||||
| * | | hostkeys: update docstring for autdodoc per #2173 | Alex Chavkin | 2023-02-16 | 1 | -1/+2 |
| | | | |||||
| * | | Changelog for PR #2173 | Alex Chavkin | 2023-02-16 | 1 | -0/+3 |
| | | | |||||
| * | | Double quotes to pass style check | Alex Chavkin | 2023-02-16 | 1 | -1/+1 |
| | | |