Skip to content
This repository was archived by the owner on Nov 23, 2024. It is now read-only.

Commit 39ac8d5

Browse files
committed
Cleanup authentication methods
Cookie authentication now used for dashboard only. Basic (legacy) authentication left only for admin access end-points.
1 parent 62b0a16 commit 39ac8d5

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/Web/Web/Controllers/AdminController.cs

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
namespace CodeMooc.Web.Controllers {
1313

1414
[Route("admin")]
15-
[Authorize(Policy = Startup.BasicAdministratorsPolicyName)]
15+
[Authorize(Policy = Startup.LegacyBasicAdministratorsPolicyName)]
1616
public class AdminController : Controller {
1717

1818
protected DatabaseManager Database { get; }

src/Web/Web/Startup.cs

+4-4
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public Startup(IConfiguration configuration) {
2121

2222
public const string AdministratorsOnlyPolicyName = "AdministratorsOnly";
2323
public const string MembersOnlyPolicyName = "MembersOnly";
24-
public const string BasicAdministratorsPolicyName = "BasicAdministratorsOnly";
24+
public const string LegacyBasicAdministratorsPolicyName = "BasicAdministratorsOnly";
2525

2626
public const string AdministratorRole = "Administrator";
2727
public const string MemberRole = "Member";
@@ -61,19 +61,19 @@ public void ConfigureServices(IServiceCollection services) {
6161
new AuthorizationPolicyBuilder()
6262
.RequireAuthenticatedUser()
6363
.RequireRole(AdministratorRole)
64-
.AddAuthenticationSchemes(CookieAuthenticationDefaults.AuthenticationScheme, BasicAuthenticationSchemeOptions.SchemeName)
64+
.AddAuthenticationSchemes(CookieAuthenticationDefaults.AuthenticationScheme)
6565
.Build()
6666
);
6767
opt.AddPolicy(
6868
MembersOnlyPolicyName,
6969
new AuthorizationPolicyBuilder()
7070
.RequireAuthenticatedUser()
7171
.RequireRole(MemberRole)
72-
.AddAuthenticationSchemes(CookieAuthenticationDefaults.AuthenticationScheme, BasicAuthenticationSchemeOptions.SchemeName)
72+
.AddAuthenticationSchemes(CookieAuthenticationDefaults.AuthenticationScheme)
7373
.Build()
7474
);
7575
opt.AddPolicy(
76-
BasicAdministratorsPolicyName,
76+
LegacyBasicAdministratorsPolicyName,
7777
new AuthorizationPolicyBuilder()
7878
.RequireAuthenticatedUser()
7979
.RequireRole(AdministratorRole)

0 commit comments

Comments
 (0)