Php-graph-sdk: Enforce HTTPS

[shengfa]

I'm using these code:
$helper = $this->facebook->getRedirectLoginHelper();
$accessToken = $helper->getAccessToken();

In getAccessToken function has $redirectUrl = FacebookUrlManipulator::removeParamsFromUrl($redirectUrl, ['code', 'state']);
If my Facebook App has turn on "Enforce HTTPS" mode, this $redirectUrl will be something like https://donmain.com/facebook/callback?enforce_https=1
This will cause Facebook return an error about "domain" or "Valid OAuth Redirect URIs".

Shouldn't remove it?

[gianpaolodn]

Login via facebook suddenly stopped working on our website yesterday night. Had to disable "enforce https" in the setting panel as a temporary work around.

this is the error you get:
Error validating verification code. Please make sure your redirect_uri is identical to the one you used in the OAuth dialog request

[BloodElf]

Same problem, how can i fix it?

[gianpaolodn]

@BloodElf if you can't disable "enforce_https" in the app setting panel, modify the code:

$redirectUrl = FacebookUrlManipulator::removeParamsFromUrl($redirectUrl, ['code', 'state']);

to

$redirectUrl = FacebookUrlManipulator::removeParamsFromUrl($redirectUrl, ['code', 'state', 'enforce_https']);

in this class: FacebookRedirectLoginHelper at line 226.

It should workaround the issue, even tho is fb that should NOT send that param at all

[BloodElf]

@gianpaolodn It works! Thank you very much!

[70nyIT]

confirm, same issue. Solved the same way, adding _'enforce_https'_

[yguedidi]

@gianpaolodn thanks! Can you open a PR with the fix?

[yguedidi]

@gianpaolodn in fact, there is already a PR for that: #1054