Skip to content

Commit 9932d7f

Browse files
authored
Merge pull request #395 from danschultzer/delete-only-if-session-key-exists
Only delete the session key if value exists
2 parents 1eedc64 + 316aac7 commit 9932d7f

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

lib/pow/plug/session.ex

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -195,12 +195,17 @@ defmodule Pow.Plug.Session do
195195
@impl true
196196
@spec delete(Conn.t(), Config.t()) :: Conn.t()
197197
def delete(conn, config) do
198-
key = client_store_fetch(conn, config)
199-
{store, store_config} = store(config)
198+
case client_store_fetch(conn, config) do
199+
nil ->
200+
conn
200201

201-
store.delete(store_config, key)
202+
key ->
203+
{store, store_config} = store(config)
202204

203-
client_store_delete(conn, config)
205+
store.delete(store_config, key)
206+
207+
client_store_delete(conn, config)
208+
end
204209
end
205210

206211
# TODO: Remove by 1.1.0

0 commit comments

Comments
 (0)