Brightspace / D2L.Security.OAuth2

Brightspace OAuth 2.0 for C#
Apache License 2.0
7 stars 16 forks source link

Split ID2LPrincipal #24

Closed j3parker closed 8 years ago

j3parker commented 8 years ago

The OAuth2 specific stuff should be factored out of ID2LPrincipal so that we can make ID2LPrincipal more sanely injectible in non-OAuth2.0 cases.

The namespace is OAuth2 specific which is a bit weird but better to get the split right so that the eventual refactor will/would be easier.

j3parker commented 8 years ago

Best to wait until after #13

j3parker commented 8 years ago

On second thought, probably not worth the churn yet (too nebulous.)