Skip to content

Commit 2f405da

Browse files
committed
fix: Session::stop() does not destory the session
1 parent 38840b0 commit 2f405da

1 file changed

Lines changed: 3 additions & 11 deletions

File tree

system/Session/Session.php

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -272,21 +272,13 @@ public function start()
272272
}
273273

274274
/**
275-
* Does a full stop of the session:
275+
* Destroys the current session.
276276
*
277-
* - destroys the session
278-
* - unsets the session id
279-
* - destroys the session cookie
277+
* @deprecated Use destroy() instead.
280278
*/
281279
public function stop()
282280
{
283-
setcookie(
284-
$this->sessionCookieName,
285-
session_id(),
286-
['expires' => 1, 'path' => $this->cookie->getPath(), 'domain' => $this->cookie->getDomain(), 'secure' => $this->cookie->isSecure(), 'httponly' => true]
287-
);
288-
289-
session_regenerate_id(true);
281+
$this->destroy();
290282
}
291283

292284
/**

0 commit comments

Comments
 (0)