Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Extract common OAuth2 middleware base class #49

Closed
Tratcher opened this issue Aug 25, 2014 · 1 comment
Closed

Extract common OAuth2 middleware base class #49

Tratcher opened this issue Aug 25, 2014 · 1 comment
Assignees
Milestone

Comments

@Tratcher
Copy link
Member

Facebook, MicrosoftAccount, Google, all use OAuth2 and should derive from an OAuth package with shared base classes for middleware, notifications, notification contexts, etc..

Specific items of concern: Shared base class for *AutehnticatedContext with shared OAuth fields like AccessToken, RefreshToken, Id, etc.. See @blowdart for specific fields.

Stretch: Can this work for Twitter also? They use OAuth1 but many of the shared fields are the same.

FYI: There are also all of these 3rd party implementations that could be simplified:
https://github.com/owin-middleware/OwinOAuthProviders/tree/master/Owin.Security.Providers

@kevinchalet
Copy link
Contributor

Here's the PR: #56 😄

Tratcher added a commit that referenced this issue Sep 8, 2014
@Tratcher Tratcher added this to the 1.0.0-alpha4 milestone Sep 8, 2014
@Tratcher Tratcher removed the 3 - Done label Oct 20, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants