-
Notifications
You must be signed in to change notification settings - Fork 15
remove pgpy support, point to pyac (separate pgpy effort) #19
Conversation
- the tests on py27 are currently broken due to the long pending issue SecurityInnovation/PGPy#217 - this autocrypt repository is not going to drop the dependency on and default of gpg or gpg2 any time soon (stability, missing keyring support) and there are a number of upcoming refactorings where all not-used code requires more effort to keep working. - there is a separate "pyac" effort aiming for implementing autocrypt with pgpy https://github.com/juga0/pyac and it -- it probably makes sense to try merge/converge between the two efforts once pgpy is usable and stable enough.
It makes sense to me to remove pgpy here if pyac is not going to be merged any time soon and then it also looks good to me the PR. If both projects are going to be merged soon, it'd make more sense pyac get reviewed and the code that uses pgpy here be replaced by pyac. Note that so far, pyac is not even included in autocrypt organization just because i don't think we have a policy for that. As a separated question that i'm not sure were should be formulated. Should it be included?
Can we point out were pgpy is not stable enough in the context of py-autocrypt?
pyac dropped support for py27, so that bug (3 months old) doesn't affect pyac. Is there any reason to support py27 in py-autocrypt or pyac?
If py-autocrypt and pyac are going to be merged, gpg(2) dependency should then stay. I'd wait for others opinion on this. Thanks. |
thanks for your review and comments. a few reply points:
summary: My priority with python autocrypt development lies with getting a stable tested package that i can use myself and others can and do use reliably for their own mail autocrypt integrations. Maybe it's time for py-autocrypt (and other autocrypt implementations) to move out of the autocrypt organization. see also autocrypt/autocrypt#320 . |
ok, i guess we don't have a way to "measure" this
ok, this seems then this is the main issue for you with
I don't think so, just because I thought that
Right, the downside is to have to mantain yet another gpg wrapper.
Yeah, it's a matter of implementing more tests.
Thanks, i'll have a look.
See question above. |
i'd like py-autocrypt to integrate with my own mutt mail setup to seemlessly handle my encryption/decryption instead of the cumbersome current mutt/crypto stuff in my config, and also as something that can be used for server-side software that so far only does clear-text mail handling (e.g. mailman3), so doesn't have its own crypto facilities that py-autocrypt could make use of. py-autocrypt also does not implement crypto itself, but delegates it to gpg (a common approach with many MUAs) so integrating with that seems like a safe option. I'd feel uneasy advertising a library like py-autocrypt as an example L1 compliant implementation if i wouldn't use it somewhat for real (even if mutt has a limited userbase, i did get questions from serveral folks at 34c3 for how to do it). |
I think mutt integration does not have anything to do with removing pgpy support (title of this PR). |
my previous comment tried to answer your question "I thought that py-autocrypt was not inteded to be an "end-user" ready tool, but rather an "example" implementation for developers. Am i wrong?.". I indeed use py-autocrypt for my setup and others consider it too, so it's not just an example implementation and that means that its crypto integration needs to be solid, process real-world stuff. See also what i posted with #32 related to the same topic. |
there is a separate "pyac" effort aiming for implementing autocrypt
with pgpy https://github.com/juga0/pyac and it probably makes
sense to try merge/converge between the two efforts once pgpy is
usable and stable enough there.
the tests on py27 are currently broken due to the
long pending issue new version of six (1.11.0) breaks pgpy with python2.7 SecurityInnovation/PGPy#217
this autocrypt repository is not going to drop the dependency on
and default of gpg or gpg2 any time soon (stability, missing keyring support)
and there are a number of upcoming refactorings where i'd like to avoid
having to refactor unused code (which anyway seems further developed with pyac)