Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

cannot delete fb cookie #60

Closed
wants to merge 1 commit into from

3 participants

@softwarevamp

remove the domain to delete cookie

@softwarevamp softwarevamp cannot delete fb cookie
remove the domain to delete cookie
c296af6
@mknwebsolutions

Wouldn't you want to include the base domain? Say you're working off a subdomain and your session is passed through to a other subdomain (i.e. ecommerce platform), this will still hold session for X domain.

@daaku

The right way to delete the cookie would be specify the domain when you delete it. The domain is necessary for when subdomains are involved.

@daaku daaku closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 30, 2013
  1. @softwarevamp

    cannot delete fb cookie

    softwarevamp authored
    remove the domain to delete cookie
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  src/base_facebook.php
View
2  src/base_facebook.php
@@ -1334,7 +1334,7 @@ public function destroySession() {
unset($_COOKIE[$cookie_name]);
if (!headers_sent()) {
$base_domain = $this->getBaseDomain();
- setcookie($cookie_name, '', 1, '/', '.'.$base_domain);
+ setcookie($cookie_name, '', 1, '/');
} else {
// @codeCoverageIgnoreStart
self::errorLog(
Something went wrong with that request. Please try again.