openwisp / django-x509

Reusable django app implementing x509 PKI certificates management
http://openwisp.org
BSD 3-Clause "New" or "Revised" License
340 stars 68 forks source link

Add optional UUID based base models #9

Open nemesifier opened 6 years ago

nemesifier commented 6 years ago

All the OpenWISP 2 models have uuid based IDs except django-x509.

We need to have uuid based fields for django-x509 too, but since this app is also used by projects that are not related to openwisp, we could keep this feature optional.

Tasks:

R9295 commented 6 years ago

@nemesisdesign I'd like to do this next if it isn't reserved for GCI

nemesifier commented 6 years ago

@R9295 this is not reserved to GCI, however, we are moving on different fronts at the same time. This feature will need some more work to be used in OpenWISP.. I really want every OW module to use only UUID, but migrating existing databases will be painful.

Anyway, before jumping on this, what about making the new passphrase feature available in openwisp-controller? https://github.com/openwisp/openwisp-controller/issues/36 it's not available yet there (just ran into an issue because in an instance I mistakenly upgraded django-x509 to the current master).

R9295 commented 6 years ago

@nemesisdesign makes sense, will do :+1: