Skip to content

Conversation

@nemesifier
Copy link
Member

Aims to close #131
Close #162
Close #163
Close #174

@davidjmemmett
Copy link

davidjmemmett commented May 24, 2025

I made a start this morning on fixing some of the tests: https://github.com/davidjmemmett/django-x509/commits/PR178-fix-tests/ - this is far from complete, but I'll keep working on this in my spare time until as many of the tests are fixed (or rewritten) as I can get through.

I don't suppose the latest changes from the master branch would merge into this branch easily? It'd make it easier to merge later on

@davidjmemmett
Copy link

I'm struggling with django_x509.tests.test_cert.TestCert.test_extensions; I even went as far as re-implementing the part where it adds UnrecognizedExtension, but I couldn't get it working.

What version of cryptography has ExtensionOID._map?

@davidjmemmett davidjmemmett mentioned this pull request May 25, 2025
4 tasks
@davidjmemmett
Copy link

I've opened a Pull Request to add some tests for these changes - there are a few which are still broken, but it'll require someone with much more experience with both cryptography and django-x509 than myself.

I hope this helps.
Cheers,
David

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: To do (general)

Development

Successfully merging this pull request may close these issues.

[change] Drop pyOpenSSL in favour of cryptography

3 participants