diff options
| author | Daniel Kahn Gillmor <[email protected]> | 2018-12-04 17:29:40 +0000 |
|---|---|---|
| committer | Andre Heinecke <[email protected]> | 2018-12-05 10:46:09 +0000 |
| commit | 878a0ad01265dba5b06429276bdcc5c21fedb6f5 (patch) | |
| tree | fb7b4738150bdd91cd191ef34bb5d82cff0da735 /lang/python/src | |
| parent | python: Clarify the meaning of ctx.decrypt(verify=[]) (diff) | |
| download | gpgme-878a0ad01265dba5b06429276bdcc5c21fedb6f5.tar.gz gpgme-878a0ad01265dba5b06429276bdcc5c21fedb6f5.zip | |
python: ctx.decrypt() has problematic error handling
* lang/python/src/core.py (Context.decrypt): document odd
error-handling behavior as a potential problem to be addressed.
Signed-off-by: Daniel Kahn Gillmor <[email protected]>
Diffstat (limited to 'lang/python/src')
| -rw-r--r-- | lang/python/src/core.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lang/python/src/core.py b/lang/python/src/core.py index 3bd6f717..f3202eb1 100644 --- a/lang/python/src/core.py +++ b/lang/python/src/core.py @@ -427,6 +427,10 @@ class Context(GpgmeWrapper): results=results) if verify: + # FIXME: should we really throw BadSignature, even if + # we've encountered some good signatures? as above, once + # we hit this error, there is no way to accept it and + # continue to process the remaining signatures. if any(s.status != errors.NO_ERROR for s in verify_result.signatures): raise errors.BadSignatures(verify_result, results=results) |