Skip to content
This repository has been archived by the owner on Aug 16, 2023. It is now read-only.

Facebook sso #44

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Facebook sso #44

wants to merge 6 commits into from

Conversation

kornifex
Copy link
Contributor

Hi again,

Here is an implementation of Facebook SSO.
It has the same requirements as Facebook SDK, which are:

  • the developer has to handle callback with
    - (BOOL)application:(UIApplication *)application openURL:(NSURL *)url sourceApplication:(NSString *)sourceApplication annotation:(id)annotation { return [SimpleAuth handleCallback:url]; }
  • Facebook prerequisites (the .plist configuration) still applies as seen in Facebook's login getting started guide

It should handle most cases, but still needs to be heavily tested.
We decided that the FBSession should be closed after each opening, so the provider will fetch a new token each time it is called.
Facebook's errors are handled and passed as underlying errors, and it is left to the developer's choice to specifically handle them.

We think that it is as simple as it can be for the developer, and that you will like it. :)

I updated the podspec to include Facebook iOS SDK, but you should review it as I'm no pro with .podspec…

Julien

@calebd
Copy link
Owner

calebd commented Apr 25, 2014

Thanks for the contribution, sorry it took me so long to get to it. I can't stand the Facebook SDK and I think I'll rebuild this manually so we don't have to rely on that. I'll get to it soon!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants