pycryptodome as optional requirement
See original GitHub issueThe cryptography library used (pycryptodome) is not pure python. As such, it requires a C compiler, and can also create problems running WSGI apps unless the WSGIApplicationGroup
is set properly. It looks like this dependency is only required for scoped keys, an optional feature that is not required for all use cases.
Would it make sense to let this be optional and throw an exception if somebody tries to use scoped keys without this dependency installed? I am happy to make a pull request but just wanted to make sure this would make sense first.
Issue Analytics
- State:
- Created 6 years ago
- Comments:9 (6 by maintainers)
Top Results From Across the Web
python/optional-requirements.txt at master - GitHub
A Python 3 asyncio Matrix framework. Contribute to mautrix/python development by creating an account on GitHub.
Read more >PyCryptodome Documentation - Read the Docs
3. Private exponent d. Only required if the key is private. 4. First factor of n (p). Optional, but the other factor q...
Read more >Developers - pycryptodome as optional requirement - - Bountysource
The cryptography library used (pycryptodome) is not pure python. As such, it requires a C compiler, and can also create problems running WSGI...
Read more >Optional dependencies - mautrix-bridges
To install specific optional dependencies, install the packages listed "Required packages" from the table of optional dependencies below.
Read more >optional-requirements.txt ... - mau.dev
python-magic ruamel.yaml SQLAlchemy commonmark lxml asyncpg aiosqlite prometheus_client setuptools uvloop python-olm unpaddedbase64 pycryptodome ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Given that we’ve deprecated Scoped Keys, I think I’ll completely drop this requirement for the next version and note that the library must be manually installed in the section that talks about the (deprecated) Scoped Keys feature.
Sorry for the delay on this. I’m still working through 1 PR and another branch soon-to-become a PR. If you want to write up a patch here, we can just deal with the merge conflicts later rather than forcing you to wait for my timetable.