Signature validation should support validating with an old secret

Offen
#101 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
35/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
api, authentication, security

Rechercherichtung

Im Issue werden keine Dateien oder Tests genannt. Beginne damit, den vorhandenen Einstiegspunkt für die OAuth2-Signaturvalidierung zu finden und seine Konfiguration und sein Verhalten mit der in der verlinkten Dokumentation zur Rotation von Zugangsdaten beschriebenen Webhook-Validierung zu vergleichen. Als abgeschlossen gilt die Aufgabe, wenn die Validierung Signaturen akzeptiert, die entweder mit dem aktuellen oder dem alten API secret erzeugt wurden, und dabei das bisherige Verhalten mit einem einzelnen Secret erhalten bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

feature request

Problem

Oauth2 signature validation will fail when during credential rotation, since the signature is generated with the oldest secret, and validation can only be configured to validate against a single secret in shopify_python_api.

Solution

This needs to be handled similar to webhook validation, where it must be possible to specify the old API secret as well as the new one for signature validation, and accept the signature if it matches the ones generated with either secret.

Vorherrschende Sprache
Python
Sterne
1.4k
Forks
388
Ø Merge
5 Std. 39 Min.
Gemergte PRs (30 T.)
1

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Shopify/shopify_python_api

Alle Issues in Shopify/shopify_python_api

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.