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

Use more DI for Cookie options #64

Closed
HaoK opened this issue Sep 18, 2014 · 3 comments
Closed

Use more DI for Cookie options #64

HaoK opened this issue Sep 18, 2014 · 3 comments

Comments

@HaoK
Copy link
Member

HaoK commented Sep 18, 2014

ICookieManager, IAuthenticationSessionStore, INotifications should all provide something like:

    //options.UseNotification<GoogleAuthenticationNotifications>();
    //options.UseNotification(new GoogleAuthenticationNotifications());
    //options.UseNotification(services => return google);

Also Consider wrapping this pattern in some sugar struct

@lodejard
Copy link

Syntax sugar would be specifically to have a value-type that can be used as options.Notification to hold the delegate, and also to hold onto the delegate. Would that go in the options repo?

@HaoK
Copy link
Member Author

HaoK commented Sep 19, 2014

Filed aspnet/Options#11 for the sugar in options

@HaoK
Copy link
Member Author

HaoK commented Oct 8, 2014

Merged 3426034

@HaoK HaoK closed this as completed Oct 8, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants