search

medispring / icure-backend

OSS version of the iCure Medical File Management software
GNU General Public License v2.0
0 stars 1 forks source link

merge upstream #83

Closed antoinepairet closed 5 years ago

antoinepairet commented 5 years ago

@cebamps Revoir la partie crypto des attachments

En particulier, il faut reviewer ceci

phil1618 commented 5 years ago

@antoinepairet Ca m'a l'air ok. J'ai juste un doute sur l'ajout des EncryptionKey dans la facade de traitement des messages. Je vais verifier que ce n'est pas breaking en faisant tourner cette version en local

phil1618 commented 5 years ago

@antoinepairet @aduchate Je remarque que le CodeDAOImpl.kt utilise le queryViewWithKeys du CouchDbICureConnector (https://github.com/antoinepairet/icure-backend/blob/39ecc8b31e831dacf5d25ae372f66fa0f78c6487/src/main/kotlin/org/taktik/icure/dao/impl/CodeDAOImpl.kt#L103) mais cette propriété a été retirée (src/main/java/org/taktik/icure/dao/impl/ektorp/CouchDbICureConnector.java)

https://github.com/antoinepairet/icure-backend/pull/83/files#diff-e6ccc0f10f54f556d8df3e114fc79858L33

cebamps commented 5 years ago

OK pour moi aussi. J'espère que le comportement par défaut pour les attachments est de ne rien déchiffrer si la clé de chiffrement est null. À tester avec un genloc.