Bug Bounty Report: remaining typos in natspec at 2.6.1 (wich/whitch/intefraces/suported/overriden)
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 25/100
- Tipo di issue
- Documentazione
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- solidity
- Ambito
- documentation
Direzione di ricerca
The typos sit in natspec comments in src/contracts/tokens/nf-token-enumerable.sol, nf-token-metadata.sol, utils/supports-interface.sol, ownership/ownable.sol and mocks/nf-token-metadata-enumerable-mock.sol. Run the grep command from the issue to confirm the locations, then check PR #302, which is already open against this issue, before starting. Done means every listed misspelling is corrected in comments only, with no code changes, and the grep returns nothing.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Found while reviewing src/contracts under the bug bounty program (latest release, @nibbstack/erc721 2.6.1). Typos are explicitly listed in the program scope.
Remaining misspellings in natspec/comments at tag 2.6.1:
| File | Line | Text | Should be |
|---|---|---|---|
src/contracts/tokens/nf-token-enumerable.sol |
151 | "Address from wich we want to remove the NFT" | which |
src/contracts/tokens/nf-token-enumerable.sol |
181 | "Address to wich we want to add the NFT" | which |
src/contracts/tokens/nf-token-metadata.sol |
83 | "an internal function that can be overriden" | overridden |
src/contracts/utils/supports-interface.sol |
14 | "Mapping of supported intefraces" | interfaces |
src/contracts/utils/supports-interface.sol |
27 | "which interfaces are suported by this contract" | supported |
src/contracts/ownership/ownable.sol |
5 | "authorization control whitch" | which |
src/contracts/mocks/nf-token-metadata-enumerable-mock.sol |
107, 123 | "from wich we want to remove" / "Address to wich we want to add" | which |
Reproduce with:
grep -rniE '\bwich\b|whitch|intefraces|suported|overriden' src/contracts
Suggested fix: a single pass with a spell checker over natspec blocks (ispell/aspell), as done previously in #252 for other files.
Tested against tag 2.6.1 (commit 198dec9).
- Lingua principale
- Solidity
- Stelle
- 1k
- Fork
- 391
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
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 fulldecent/erc721
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
fulldecent/erc721#299 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
fulldecent/erc721#293 · 1 reazione ·
-
Bug Bounty Report: Security Issue in ERC-721 Reference ImplementationForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 8/100
fulldecent/erc721#306 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
fulldecent/erc721#304 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
fulldecent/erc721#303 · 1 commento ·
Tutte le issue di fulldecent/erc721
Issue simili
-
documentation enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
good first issue priority:low type:docs
Difficoltà 1/5 1-3 ore Idoneità per principianti 95/100
crazy-goat/rabbit-stream#969 ·
I maintainer di solito rispondono entro 1 giorno
-
question
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
backend docs
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
Open-Locker/Open-Locker#368 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
fsprojects/FSharp.Formatting#1370 ·
I maintainer di solito rispondono entro 1 giorno