Skip to content

Commit

Permalink
fix: use unique variable name when clearing old state data to avoid s…
Browse files Browse the repository at this point in the history
…etting state data to incorrect session key
  • Loading branch information
jdeepee committed Jul 17, 2024
1 parent 0ad753c commit 01583a3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions authlib/integrations/starlette_client/integration.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,9 @@ async def set_state_data(self, session: Optional[Dict[str, Any]], state: str, da
await self.cache.set(key, json.dumps({'data': data}), self.expires_in)
elif session is not None:
# clear old state data to avoid session size growing
for key in list(session.keys()):
if key.startswith(key_prefix):
session.pop(key)
for old_key in list(session.keys()):
if old_key.startswith(key_prefix):
session.pop(old_key)
now = time.time()
session[key] = {'data': data, 'exp': now + self.expires_in}

Expand Down

0 comments on commit 01583a3

Please sign in to comment.