diff options
| author | Ruben Pollan <meskio@sindominio.net> | 2016-03-20 18:44:23 +0100 |
|---|---|---|
| committer | Ruben Pollan <meskio@sindominio.net> | 2016-03-20 20:01:01 +0100 |
| commit | 288c775034d4c18846518a11677e4580a91cf437 (patch) | |
| tree | 37f6aabd1f301693b5b013131b717ecd75fea907 | |
| parent | d44d72c816ebc668931071cf5bd8e37e0be32e05 (diff) | |
[bug] Return KeyNotFound Failure if not valid key is given to put_raw_key
- Resolves: #7974
| -rw-r--r-- | changes/next-changelog.txt | 2 | ||||
| -rw-r--r-- | src/leap/keymanager/__init__.py | 11 |
2 files changed, 10 insertions, 3 deletions
diff --git a/changes/next-changelog.txt b/changes/next-changelog.txt index 163c1ee..2dd34d4 100644 --- a/changes/next-changelog.txt +++ b/changes/next-changelog.txt @@ -21,6 +21,8 @@ Features Bugfixes ~~~~~~~~ +- `#7974 <https://leap.se/code/issues/7974>`_: Return KeyNotFound Failure if not valid key is given to put_raw_key. + - `#1235 <https://leap.se/code/issues/1235>`_: Description for the fixed stuff corresponding with issue #1235. - Bugfix without related issue number. diff --git a/src/leap/keymanager/__init__.py b/src/leap/keymanager/__init__.py index 9e3b6ee..1106c23 100644 --- a/src/leap/keymanager/__init__.py +++ b/src/leap/keymanager/__init__.py @@ -820,9 +820,10 @@ class KeyManager(object): :return: A Deferred which fires when the key is in the storage, or which fails with KeyAddressMismatch if address doesn't match - any uid on the key or fails with KeyNotValidUpdate if a key - with the same uid exists and the new one is not a valid update - for it. + any uid on the key or fails with KeyNotFound if no OpenPGP + material was found in key or fails with KeyNotValidUpdate if a + key with the same uid exists and the new one is not a valid + update for it. :rtype: Deferred :raise UnsupportedKeyTypeError: if invalid key type @@ -831,6 +832,10 @@ class KeyManager(object): _keys = self._wrapper_map[ktype] pubkey, privkey = _keys.parse_ascii_key(key, address) + + if pubkey is None: + return defer.fail(KeyNotFound(key)) + pubkey.validation = validation d = self.put_key(pubkey) if privkey is not None: |