GoogleCloudPlatform / GoogleCloudPlatform/appengine-php-sdk

Memcache: Honor the expiry parameter in increment() and decrement()

Offen
#116 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
PHP
Sterne
29
Forks
36
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

`incrementInternal()` accepts the documented `$expiry` argument but never uses it — `MemcacheIncrementRequest` has no expiration field — so any item created through the `initial_value` path is stored without an expiration and lives until evicted. Callers using `increment()` for rate limiting or short-lived counters accumulate immortal keys, and patterns that rely on expiry to self-heal (e.g. a worker counter that must reset after a crash) never recover.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne bei incrementInternal() und verfolge, wie das dokumentierte Argument $expiry zu einer MemcacheIncrementRequest wird. Stelle sicher, dass der initial_value-Pfad das Ablaufverhalten beibehält, und überprüfe anschließend, dass increment() und decrement() keine unsterblichen Schlüssel mehr erstellen, wenn ein Ablauf angegeben ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
memcached, php
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Ruhig
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
72/100

Neue Issues direkt in Ihr Postfach

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