Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
We belatedly updated to use facebook's PHP SDK v3.1 today, and found a few problems that were because the signed request from Facebook was not passed through to the PHP SDK.
The attached patch fixes this.
Comment | File | Size | Author |
---|---|---|---|
fb-signed-request.patch | 672 bytes | james.williams |
Comments
Comment #1
ianthomas_ukThis patch works for me
Comment #2
ianthomas_ukThis patch no longer applies because code round about it has changed, but given the patch has been sitting in the issue queue for the best part of a year I'm not sure if it is still needed or not. Can anyone who knows the API better advise?
Comment #3
Dave Cohen CreditAttribution: Dave Cohen commentedI'm not sure what the original problems were. If a change like this is needed, somebody please explain in detail. I don't think it's needed.
Comment #4
james.williams CreditAttribution: james.williams commentedWe are still using the patch in our builds, but we're also using an old dev version of the module, as that's what we have tested fully for our purposes. It does look like the signed request is now handled elsewhere in the code of the latest dev version, so perhaps it isn't needed any more. BUT I haven't tested that latest code against our builds so I can't say for certain.
@ianmthomasuk - that probably means you'll have to do some testing to know for sure.
I don't know exactly what the problems we encountered were, but I expect they were something to do with that specific version of the FB module. Here's what I wrote on our internal ticket for our client, which unfortunately isn't very technical/specific: