25/11/2022 - Meeting Notes Libs

Asistentes

• @Garcia-Alvarez Roberto

• @Juan Tavira

• @Daniel de la Sota Blanchart

• @Antonio González Sanz

• @Alejandro Alfonso

• @Alejandro Nieto Gallego

• @Former user (Deleted)

• @Iker Ruiz de Infante Gonzalez

• @Former user (Deleted)

• @Manuel Gallego Chinchilla

• @Pedro Gallego

• @Pablo Molina Martínez

PUNTOS DEL DÍA:

• Se realizó una sesión de trabajo el día Nov 24, 2022 convocada por @Garcia-Alvarez Roberto en la que participaron @Iñigo Garcia , @Juan Tavira , @Carlos Pastor Matut , @Daniel de la Sota Blanchart , @Manuel Gallego Chinchilla y @Garcia-Alvarez Roberto y se tomaron la siguientes decisiones:

  • AlastriaPublicKeyRegistry.sol

    • Se va a realizar sobrecarga en todos los métodos para las nuevas funciones cuyo parámetro sea el hash de la clave pública.

    • No se renombra la función de addKey para que los proyectos donde se este utilizando los smart contracts y la librería de Alastria no tengan que cambiar nada. La nueva función de addPublicKey es la única función que no va a tener sobrecarga ya que se va a unificar la nomenclatura con el resto de funciones del Smart Contracts.

    • Se elimina el mapping que contiene la lista de los hashes de las claves públicas, ya que no se quiere guardar un histórico de las claves de los usuarios.

    • La función addPublicKey (la nueva) ya no revoca la clave pública previa porque ya no se puede consultar cual era la anterior clave pública, por tanto, ahora es el usuario el que debe de revocar la clave pública primero antes de añadir una nueva. Además, se permite que el usuario pueda tener más de una clave pública en uso.

• @Manuel Gallego Chinchilla ha llevado a cabo los cambios pertienentes con respecto a las decisiones tomadas en la sesión de trabajo. Quedaría pendiente la subida de la versión del comilador de solidity a la 0.8.17 ya que se ha probado y da errores, con lo cual se ha decidido realizar esta subida en otra rama cuando tengamos esta versión de los smart contracts terminada y funcionando correctamente.

• Faltaría revisar los smart contracts, incluidos comentarios y realizar los tests unitarios para esta nueva versión del AlastriaPublicKeyRegistry.sol. Una vez se tengan los smart contracts estables y probados se pasaría a la parte de librerías y por último al repositorio de ejemplos para crear nuevos ejemplos con las nuevas funciones que trabajan con hashes.

• @Juan Tavira explica la decisión tomada en el equipo de arquitectura de unificar los dos registries el de credenciales y el de presentaciones con el objetivo de ofuscar mas la información y de que no se sepa cuando se esta registrando la emisión/revocación de una credencial o de una presentación. De esta manera conseguimos que la información que se registre este más ofuscada ya que solo los intervinientes de esa emision/revocación son los que saben realmente de que se esta hablando mientras que el resto de participantes de la red, no saben de que se trata. Para ello hay que unificar los estados de las credenciales y presentaciones para que el estado de ambos objetos sean, en lugar de un enum un numérico. Esto se va a acometer en la misma versión que los cambios al publicKeyRegistry para tener una versión con un empaque mucho mayor.

• @Garcia-Alvarez Roberto crea la tarjeta (https://github.com/alastria/alastriaID-truffle-contracts/issues/62 ) para acometer la unificación de los dos registries. @Juan Tavira completará con mayor detalle lo que hay que hacer en esta tarea.

• La sesión del viernes Dec 9, 2022 se cancela por puente.