Skip to content

Breaking change in AddAuthorization(o =>) overload lives in a different assembly #14525

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

Closed
ajaybhargavb opened this issue Sep 19, 2019 · 2 comments · Fixed by #15044
Closed
Assignees
Labels
breaking-change Indicates a .NET Core breaking change

Comments

@ajaybhargavb
Copy link

ajaybhargavb commented Sep 19, 2019

Original announcement aspnet/Announcements#386

Breaking change in AddAuthorization(o =>) overload lives in a different assembly

We renamed the core AddAuthorization methods that used to live in Microsoft.AspNetCore.Authorization to be AddAuthorizationCore, the old AddAuthorization methods still exist, but are in the Microsoft.AspNetCore.Authorization.Policy package instead. Apps that are using both should see no impact, but apps that were not using the policy package will need to switch to using AddAuthorizationCore

Version introduced

3.0

Old behavior

AddAuthorization methods existed in Microsoft.AspNetCore.Authorization

New behavior

AddAuthorization methods exist in Microsoft.AspNetCore.Authorization.Policy, AddAuthorizationCore is the new name for the old methods

Reason for change

We wanted the better AddAuthorization name to add all of the common services needed for authorization, so we decided to take over the old name in 3.0..

Recommended action

Either add a reference to Microsoft.AspNetCore.Authorization.Policy or use AddAuthorizationCore instead

Category

ASP.NET Core

Affected APIs

AddAuthorization(Action<AuthorizationOptions>)


Issue metadata

  • Issue type: breaking-change
@dotnet-bot dotnet-bot added breaking-change Indicates a .NET Core breaking change ⌚ Not Triaged Not triaged labels Sep 19, 2019
@ajaybhargavb
Copy link
Author

Closing as we won't need these here.

@mairaw mairaw removed ⌚ Not Triaged Not triaged breaking-change Indicates a .NET Core breaking change labels Sep 20, 2019
@Pilchie
Copy link
Member

Pilchie commented Oct 1, 2019

Re-opening based on current plan to add these to dotnet/docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Indicates a .NET Core breaking change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants