Rename `decode` function `unsafeDecode` to highlight the risk
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- javascript, node.js
- Bereich
- security
Rechercherichtung
Beginne damit, die öffentlichen decode- und verify-Einstiegspunkte im JavaScript-Paket zu lokalisieren und alle Verweise auf decode zu prüfen. Bestätige die Kompatibilität und den Migrationsumfang für unsafeDecode, und definiere den Abschluss als konsistente Umbenennung mit aktualisierten Aufrufern und Dokumentation.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the problem you'd like to have solved
I'm a DevSec. I help developers teams find and fix their vulnerabilities.
In two of my last missions, I've seen developers using decode instead of verify even if the documentation has been improved.
Describe the ideal solution
To avoid this and reduce the vulnerable apps, I think it would be great to rename the decode function and call it unsafeDecode. (as suggested by @youssef-jbili)
Other libraries have done it and it helps developers reduce their mistakes:
dangerouslySetInnerHTMLfor ReactbypassSecurityTrustHtmlfor Angular
I can make a Pull Request if you think it is a good idea.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 18.2k
- Forks
- 1.3k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus auth0/node-jsonwebtoken
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
auth0/node-jsonwebtoken#1042 · 1 Kommentar ·
-
`jwt.sign()` callback is executed twice for "The payload already has an "..." property" errorsEvtl. vergeben @cobyfrombrooklyn-bot hat das vor 231 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
auth0/node-jsonwebtoken#1000 · 2 Kommentare · 1 Reaktion ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
auth0/node-jsonwebtoken#1048 ·
-
verify() resolves a string secret by attempting createPublicKey() first, costing 4x-52x on the HS* pathEvtl. vergeben @Hashim1999164 hat das vor 26 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 65/100
auth0/node-jsonwebtoken#1046 ·
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 10/100
auth0/node-jsonwebtoken#1034 ·
Alle Issues in auth0/node-jsonwebtoken
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
woocommerce/woocommerce-google-analytics-integration#699 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
t0ggles
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
nolimits4web/swiper#8240 · 1 Kommentar ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Inter-Actief/amelink#131 ·