Solidity: Validate user-supplied option strings at build entry points
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
- #897 di @ninglinLiu — chiusa senza merge
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- solidity, typescript
- Ambito
- blockchain, tooling
Direzione di ricerca
Inizia esaminando il branch WIP validateSolidityStrings e la relativa validazione nel package solidity. Poi ispeziona gli entry point buildERC20/buildERC721 e i package buildERC7984 e uniswap-hooks buildHooks. Il lavoro è completo quando ogni confine di build rifiuta i caratteri non validi elencati con un OptionsError prima di generare Solidity.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- TypeScript
- Stelle
- 296
- Fork
- 188
- Merge medio
- 3h 24m
- PR unite (30g)
- 11
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di OpenZeppelin/contracts-wizard
-
Stellar: Generated test asserts against the contract identifier instead of the stored token nameAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
OpenZeppelin/contracts-wizard#805 ·
I maintainer di solito rispondono entro 1 giorno
-
Document properly that yarn classic is required (v1.X)Forse già presa @0xMayurrr l’ha presa 245 giorni fa. Aperta
Difficoltà 1/5 1-3 ore Idoneità per principianti 68/100
OpenZeppelin/contracts-wizard#739 ·
I maintainer di solito rispondono entro 1 giorno
-
Solidity: Governor absolute quorum values accepted by validation generate code that does not compileForse già presa @RaphaelFakhri l’ha presa 17 giorni fa. Aperta
Difficoltà 3/5 Mezza giornata Idoneità per principianti 62/100
OpenZeppelin/contracts-wizard#895 ·
I maintainer di solito rispondono entro 1 giorno
-
Solidity: Premint amounts below 1 with a leading fractional zero generate code that does not compileForse già presa @RaphaelFakhri l’ha presa 20 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 20/100
OpenZeppelin/contracts-wizard#893 ·
I maintainer di solito rispondono entro 1 giorno
-
Follow-ups from the Chainlink CCIP and ACE integration (#880)Forse già presa @nahimterrazas l’ha presa 25 giorni fa. Aperta
OpenZeppelin/contracts-wizard#881 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di OpenZeppelin/contracts-wizard
Issue simili
-
Wallet functions throws TypeError when the wallet has no account IDForse già presa @Medhansh-741 l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
hiero-ledger/hiero-sdk-js#4781 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
dataforgoodfr/mediterranean-rescue#91 ·
I maintainer di solito rispondono entro 1 giorno
-
transformer-directives: `@apply` drops a utility whose CSS is a substring of an earlier one'sForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Add: KRT TVApertaapproved check:passed streams:add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 65/100
iptv-org/iptv#58647 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
milind-soni/OpenMausBot#5829 ·
I maintainer di solito rispondono entro 1 giorno