matrix-org / matrix-org/python-signedjson
Don't expect monkeypatched nacl.signing.SigningKey instances
Offen
Dieses Issue hat noch niemand übernommen.
enhancement
- Vorherrschende Sprache
- Python
- Sterne
- 13
- Forks
- 10
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
A nicer way to fix matrix-org/synapse#11714 (imo)
- Define a new type which holds the data
(key: nacl.signing.SigningKey, alg: str, version: str). Either inherit fromnacl.signing.SigningKeyor introduce a new wrapper. - Change functions that take a
signedjson.types.SigningKeyto accept this new data type. - Major version bump.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Funktionen zu finden, die signedjson.types.SigningKey akzeptieren, und prüfe matrix-org/synapse#11714 auf das Problem mit dem Monkeypatching. Definiere den vorgeschlagenen Schlüssel-Datentyp, aktualisiere die Schnittstellen dieser Funktionen und verifiziere, dass die API-Änderung der Hauptversion konsistent abgebildet ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- cryptography, security
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100