Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Breaking Change: ConfigureAuthentication => AddAuthentication #82

Closed
jmblakl opened this issue Oct 16, 2015 · 6 comments
Closed

Breaking Change: ConfigureAuthentication => AddAuthentication #82

jmblakl opened this issue Oct 16, 2015 · 6 comments

Comments

@jmblakl
Copy link

jmblakl commented Oct 16, 2015

I was unable to find the rename of ConfigureAuthentication to AddAuthentication in the breaking change list. Forgive me if I missed it.

@fmarcoux96
Copy link

Which authentication are you trying to configure?

Here's the methods:

In Microsoft.AspNet.Security.Cookies: IAppBuilder.ConfigureCookieAuthentication()
In Microsoft.AspNet.Security.OpenId: IAppBuilder.ConfigureOpenIdAuthentication()
In Microsoft.AspNet.Security.{Provider}: IAppBuilder.Configure{Provider}Authentication() (Like Facebook, Twitter, etc.)

If none of these is what you're searching, tell us more please.

@NeelBhatt
Copy link

It is not correct place to post issue as this repository is used for announcements.

@fmarcoux96
Copy link

@NeelBhatt If it's possible, migrate the issue. I'm not responsible for that, I just wanna help!

@aspnet aspnet locked and limited conversation to collaborators Oct 16, 2015
@HaoK HaoK changed the title Breaking Change doc miss: ConfigureAuthentication Breaking Change: ConfigureAuthentication => AddAuthentication Oct 16, 2015
@HaoK
Copy link
Member

HaoK commented Oct 16, 2015

Yeah this was removed and rolled into an overload of AddAuthentication which is the new pattern across Repos

@Eilon
Copy link
Member

Eilon commented Oct 17, 2015

@HaoK in which release was this changed? Can you set the milestone on this issue?

@davidfowl
Copy link
Member

We should make another issue with the details

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

6 participants