OCA
[18.0][MIG] l10n_es_aeat
I have tested it with https://github.com/OCA/l10n-spain/pull/4046 and so far so good. Nevertheless, I am not sure if it is worth it to integrate with the certificate module. We would lose the feature of specifying the folder were the certificate are stored in the filesystem. And also, the password to unlock the certificates is stored in the certificate.key model, whereas in the AEAT module is not stored, but the information is extracted when the user puts the password. Of course the password cannot be filled in in the migration.
I will appreciate your help and try to attend comments in there.
I marked this as ready mostly to see your opinion about the certificate integration:
- We lose the feature of specifying a folder were the certificate is stored (perhaps another module can be done, like certificate_folder) to add that this feature.
- The password do unlock the certificate is stored in the certificate.key model:
- The migration is not the typical one, I stored the information as a ir.config.parameter in order to restore it in the post-migration, but I am not convinced if this is a safe procedure or if there is a better way.
En este repositorio se puede hablar en español sin problema, Aarón.
Sobre el primer punto, creo que no es problema no definir la carpeta. De hecho, al final pones cualquier valor.
Sobre el segundo, sí me preocupa más que digas que almacenas contraseñas privadas de certificado en la BD. ¿Eso no se puede evitar?
Los scripts de migración podemos afinarlos después de ver primero esta parte.
Gracias @pedrobaeza. Creo lo más sencillo que se me ocurre es hacer un módulo certificate_encrypt que convierta los campos de password a metodos compute y guarde las password encriptada como fields.Binary() ¿Crees que puede funcionar?
¿ @AaronHForgeFlow @pedrobaeza se está avanzando algo con la migración del módulo?
Creo que la implmentación o no con el módulo certificates no debería retrasar la migración.
Aun así, apoyandome en https://github.com/OCA/l10n-spain/issues/4100#issuecomment-2890936427 creo que sería mejor adoptar certificates.
Hola @EmilioPascual , de momento está parada, pero las pruebas que había hecho hasta el momento me habían ido bien.
Ahora mismo sigue manejando los certificados tal y como estaban en v17. Quizás sea un buen tema para trabajar en los OCA Days la semana que viene.
Topic incluido en https://docs.google.com/document/d/1LqeAxG9GmH3tmtNgTyOImbYAQz_ByXEBounaTk4om_4/edit?tab=t.0#heading=h.59kau1lu1g3a
Tened en cuenta este PR de la 17.0: https://github.com/OCA/l10n-spain/pull/4212
Saludos
This PR looks fantastic, let's merge it! Prepared branch 18.0-ocabot-merge-pr-4033-by-pedrobaeza-bump-nobump, awaiting test results.
Congratulations, your PR was merged at a01c2e33e75b59e4cdc26dd750e56242e3084ed0. Thanks a lot for contributing to OCA. ❤️