fix: seed roles

CarlosPavajeau · Jun 26, 2021 · 3e2ebdb · 3e2ebdb
1 parent c9618a2
commit 3e2ebdb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/Armory.Api/Extensions/Infrastructure.cs b/src/Armory.Api/Extensions/Infrastructure.cs
@@ -132,10 +132,8 @@ public static IApplicationBuilder ConfigureCors(this IApplicationBuilder app)
             return app;
         }
 
-        public static IApplicationBuilder SeedRoles(this IApplicationBuilder app)
+        public static IApplicationBuilder SeedRoles(this IApplicationBuilder app, RoleManager<ArmoryRole> roleManager)
         {
-            var roleManager = app.ApplicationServices.GetRequiredService<RoleManager<ArmoryRole>>();
-
             if (!roleManager.RoleExistsAsync("Developer").Result)
             {
                 var developerRole = new ArmoryRole {Name = "Developer"};

diff --git a/src/Armory.Api/Startup.cs b/src/Armory.Api/Startup.cs
@@ -1,6 +1,8 @@
 using Armory.Api.Extensions;
+using Armory.Users.Domain;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
+using Microsoft.AspNetCore.Identity;
 using Microsoft.Extensions.Configuration;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
@@ -27,7 +29,7 @@ public void ConfigureServices(IServiceCollection services)
         }
 
         // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
-        public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
+        public void Configure(IApplicationBuilder app, IWebHostEnvironment env, RoleManager<ArmoryRole> roleManager)
         {
             app.ConfigureCors();
 
@@ -44,7 +46,7 @@ public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
             app.UseAuthentication();
             app.UseAuthorization();
 
-            app.SeedRoles();
+            app.SeedRoles(roleManager);
 
             app.UseEndpoints(endpoints => { endpoints.MapControllers(); });
         }