# Acciones por Auditoría de Coinspect

Se han realizado correcciones sobre varios de los issues que nos han detectado en la audtoria.

Adicionalmente hemos tomado todos los consejos que nos sugirieron sobre las versiones y testing.

Siguiendo sus recomendaciones, también hemos creado una vía de escape en la cual se podrá llamar a una futura función del contrato de staking de ethereum con la previa aprobación de los usuarios. Esto en caso de que futuros cambios en el hard-fork de ethereum hagan nuestro contrato incompatible, lo cual es poco probable pero no deja de ser un factor incierto.

## <span style="color:green">SNS-01(hecho)</span> Platform owner can extend exitDate for an indefinite period

Se han tomado medidas para este punto. Se ha limitado el tiempo futuro para cada petición en cuestión.
A parte se ha implementado una multisig para que el operador y el owner tengan que participar de la decisión.
github: github: https://github.com/Sensei-Node/SenseiStake/commit/eb19d2ca39d25f4523ffb55f67dbef1f3f727959

## <span style="color:green">SNS-02 (hecho)</span> Platform owner can force users into accepting a higher commission rate

Se ha eliminado la funcionalidad de cambio de commision rate.

github: github: https://github.com/Sensei-Node/SenseiStake/commit/eb19d2ca39d25f4523ffb55f67dbef1f3f727959

## <span style="color:green">SNS-03 (hecho)</span> Invalid BLS signature can lead to funds loss

Se ha agregado unas verificaciones extras para evitar este tipo de inconveniente.

Github: https://github.com/Sensei-Node/SenseiStake/commit/c1d95ce1a70b59d7a75d58c5a98b2269ff759f1c

## <span style="color:green">SNS-04 (hecho)</span> Users’funds can be lost after refunding validator

Se ha agregado el mapping que recomiendan para almacenar los pubkeys utilizados.

github: github: https://github.com/Sensei-Node/SenseiStake/commit/eb19d2ca39d25f4523ffb55f67dbef1f3f727959

## <span style="color:orange"> SNS-05 </span>(no hemos tomado medidas) Deposit to pre funded validator will cause funds loss

No se ha realizado acciones en estes punto. Consideramos que no es muy probable que pueda suceder este tipo de accion. El unico que puede hacer el front run además es el owner, el cual está resguardado por SenseiNode en una multifirma.

## <span style="color:green">SNS-06 (hecho)</span> NFT key parameters not displayed on marketplaces

Se han agregado los campos requeridos en como metadada del nft.

github: https://github.com/Sensei-Node/SenseiStake/commit/eb19d2ca39d25f4523ffb55f67dbef1f3f727959

## <span style="color:green">SNS-07 (hecho)</span> Check ERC721 allowance instead of ownership

Se ha realizado la recomendación sugerida.

github: https://github.com/Sensei-Node/SenseiStake/commit/eb19d2ca39d25f4523ffb55f67dbef1f3f727959

## <span style="color:orange">SNS-08 (sin acciones tomadas)</span> Users can fund an unexpected validator with different conditions

No se ha realizado acciones en estes punto ya que se considera que es un caso de impacto casi nulo.

## <span style="color:green">SNS-09 (hecho)</span> Hardcoded value (32 ETH)

Se ha cambiado segun la recomendación sugerida.

## <span style="color:green">SNS-10 (hecho)</span> 404 on tokenURI link

Se ha cambiado el link para que apunte al sitio correcto