Skip to content
This repository was archived by the owner on Jan 13, 2022. It is now read-only.

Commit 583016d

Browse files
author
Fosco Marotto
committed
Merge pull request #112 from SammyK/enable-app-secret-proof-by-default
Enabled app secret proof by default
2 parents d8b2bcb + 4be7024 commit 583016d

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

src/Facebook/FacebookSession.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ class FacebookSession
5555
/**
5656
* @var bool
5757
*/
58-
private static $useAppSecretProof = false;
58+
private static $useAppSecretProof = true;
5959

6060
/**
6161
* When creating a Session from an access_token, use:

tests/FacebookRequestTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,8 @@ public function testGracefullyHandlesUrlAppending()
120120

121121
public function testAppSecretProof()
122122
{
123+
$enableAppSecretProof = FacebookSession::useAppSecretProof();
124+
123125
FacebookSession::enableAppSecretProof(true);
124126
$request = new FacebookRequest(
125127
FacebookTestHelper::$testSession,
@@ -135,7 +137,9 @@ public function testAppSecretProof()
135137
'GET',
136138
'/me'
137139
);
138-
$this->assertTrue(!isset($request->getParameters()['appsecret_proof']));
140+
$this->assertFalse(isset($request->getParameters()['appsecret_proof']));
141+
142+
FacebookSession::enableAppSecretProof($enableAppSecretProof);
139143
}
140144

141145
}

0 commit comments

Comments
 (0)