Check alpha scope in IR

Abierto Apto para principiantes
#4,280 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
69/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
go

Línea de trabajo

Busca validateCosigners() en el cliente morph y compara su ámbito de witness para direcciones alpha con GetUniversalSignerScope(). El issue también indica que se elimine la regla de ámbito de netmap-to-container, que ya no es necesaria tras neofs-contract#532. La tarea estará terminada cuando ambas comprobaciones de ámbito coincidan con las restricciones esperadas y se elimine la regla obsoleta.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

I4 neofs-storage S3 security U4

Is your feature request related to a problem? Please describe.

I'm always frustrated when we don't check some inputs. validateCosigners() in morph client checks for alpha address, but doesn't check its witness scope.

Describe the solution you'd like

Check that it's the same scope as in GetUniversalSignerScope().

Additional context

It doesn't matter much practically since all contracts are known in FS chains and we ensure only valid calls are approved, but still it's better to check for expected constraints.

Also note that netmap->container scope is no longer needed after https://github.com/nspcc-dev/neofs-contract/pull/532, it can be deleted from scope rules list.

Lenguaje dominante
Go
Estrellas
37
Forks
54
Merge medio
1 d 6 h
PR fusionados (30 d)
41

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de nspcc-dev/neofs-node

Todos los issues de nspcc-dev/neofs-node

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.