Page MenuHomePhabricator

Remove Facebook OAuth dependency on "security_settings" property
ClosedPublic

Authored by epriestley on Feb 19 2021, 6:25 PM.
Tags
None
Referenced Files
F18571642: D21571.id51346.diff
Wed, Sep 10, 5:14 AM
F18571638: D21571.id51345.diff
Wed, Sep 10, 5:14 AM
F18519470: D21571.diff
Fri, Sep 5, 5:24 PM
F18418315: D21571.id.diff
Sat, Aug 30, 1:05 PM
F18418114: D21571.id51345.diff
Sat, Aug 30, 12:42 PM
F18417232: D21571.id51346.diff
Sat, Aug 30, 11:31 AM
F18154643: D21571.id51345.diff
Aug 14 2025, 8:23 PM
F18112920: D21571.id.diff
Aug 12 2025, 9:55 PM
Subscribers
None

Details

Summary

Ref T13615. This property was removed from the Facebook API at some point, perhaps November 2020. Stop relying no it.

Test Plan

Created a local Facebook OAuth app, registered a new account locally.

Diff Detail

Repository
rP Phabricator
Branch
fb1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25154
Build 34718: Run Core Tests
Build 34717: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 19 2021, 6:27 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.