Solidity: Validate user-supplied option strings at build entry points
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
- #897 de @ninglinLiu — cerrado sin fusionar
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- solidity, typescript
- Área
- blockchain, tooling
Línea de trabajo
Comienza revisando la rama WIP validateSolidityStrings y su validación en el paquete solidity. Después, inspecciona los puntos de entrada buildERC20/buildERC721 y los paquetes buildERC7984 y uniswap-hooks buildHooks. La tarea estará terminada cuando cada límite de build rechace los caracteres no válidos indicados con un OptionsError antes de generar Solidity.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Some characters in user-supplied option strings (name, symbol, uri) can't round-trip cleanly through generated Solidity contracts — ASCII control characters, Unicode line terminators recognised by Solidity's lexer (NEL, LS, PS, VT, FF), and lone UTF-16 surrogates. Today, some of these slip through and produce non-compiling Solidity or contracts that store unexpected bytes.
Add input validation at each build entry point (buildERC20, buildERC721, etc.) to reject these characters with an OptionsError, so the failure surfaces at the build boundary rather than downstream.
A WIP here covers the solidity package. The confidential (buildERC7984) and uniswap-hooks (buildHooks) packages need the same treatment.
- Lenguaje dominante
- TypeScript
- Estrellas
- 296
- Forks
- 188
- Merge medio
- 3 h 24 min
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de OpenZeppelin/contracts-wizard
-
Stellar: Generated test asserts against the contract identifier instead of the stored token nameAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
OpenZeppelin/contracts-wizard#805 ·
Los mantenedores suelen responder en 1 día
-
Document properly that yarn classic is required (v1.X)Posiblemente ocupada @0xMayurrr la tomó hace 245 días. Abierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 68/100
OpenZeppelin/contracts-wizard#739 ·
Los mantenedores suelen responder en 1 día
-
Solidity: Governor absolute quorum values accepted by validation generate code that does not compilePosiblemente ocupada @RaphaelFakhri la tomó hace 17 días. Abierto
Dificultad 3/5 Medio día Aptitud para principiantes 62/100
OpenZeppelin/contracts-wizard#895 ·
Los mantenedores suelen responder en 1 día
-
Solidity: Premint amounts below 1 with a leading fractional zero generate code that does not compilePosiblemente ocupada @RaphaelFakhri la tomó hace 20 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 20/100
OpenZeppelin/contracts-wizard#893 ·
Los mantenedores suelen responder en 1 día
-
Follow-ups from the Chainlink CCIP and ACE integration (#880)Posiblemente ocupada @nahimterrazas la tomó hace 25 días. Abierto
OpenZeppelin/contracts-wizard#881 · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de OpenZeppelin/contracts-wizard
Issues similares
-
Wallet functions throws TypeError when the wallet has no account IDPosiblemente ocupada @Medhansh-741 la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
hiero-ledger/hiero-sdk-js#4781 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
dataforgoodfr/mediterranean-rescue#91 ·
Los mantenedores suelen responder en 1 día
-
transformer-directives: `@apply` drops a utility whose CSS is a substring of an earlier one'sPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Add: KRT TVAbiertoapproved check:passed streams:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
iptv-org/iptv#58647 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
milind-soni/OpenMausBot#5829 ·
Los mantenedores suelen responder en 1 día