Merge pull request #10 from aguacongas/feature/redis-store

Feature/redis store
Aguafrommars · Dec 21, 2018 · 0a150ef · 0a150ef
2 parents 12ae2ff + b0c90ca
commit 0a150ef
Show file tree

Hide file tree

Showing 30 changed files with 815 additions and 865 deletions.
diff --git a/.gitignore b/.gitignore
@@ -332,4 +332,6 @@ ASALocalRun/
 .mfractor/
 /ReportGenerator
 /coverage/docs
-coverage.*
+coverage.*
+
+*.xml
diff --git a/DymamicAuthProviders.sln b/DymamicAuthProviders.sln
@@ -15,6 +15,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution
 		.npmrc = .npmrc
 		.releaserc.json = .releaserc.json
 		appveyor.yml = appveyor.yml
+		appveyorinit.ps1 = appveyorinit.ps1
 		build.cmd = build.cmd
 		build.ps1 = build.ps1
 		dotnet-install.ps1 = dotnet-install.ps1
@@ -38,6 +39,10 @@ Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Aguacongas.AspNetCore.Authe
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Aguacongas.AspNetCore.Authentication.TestBase", "src\Aguacongas.AspNetCore.Authentication.TestBase\Aguacongas.AspNetCore.Authentication.TestBase.csproj", "{B3DEE02F-9CD3-4009-AFEE-00F0B09811EC}"
 EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Aguacongas.AspNetCore.Authentication.Redis", "src\Aguacongas.AspNetCore.Authentication.Redis\Aguacongas.AspNetCore.Authentication.Redis.csproj", "{CE9FE264-6139-47BF-970C-D1BBFCD7EFEA}"
+EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Aguacongas.AspNetCore.Authentication.Redis.Test", "test\Aguacongas.AspNetCore.Authentication.Redis.Test\Aguacongas.AspNetCore.Authentication.Redis.Test.csproj", "{87CFA3EF-2A15-4861-A6DC-C74E9CA3EBCE}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -68,6 +73,14 @@ Global
 		{B3DEE02F-9CD3-4009-AFEE-00F0B09811EC}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{B3DEE02F-9CD3-4009-AFEE-00F0B09811EC}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{B3DEE02F-9CD3-4009-AFEE-00F0B09811EC}.Release|Any CPU.Build.0 = Release|Any CPU
+		{CE9FE264-6139-47BF-970C-D1BBFCD7EFEA}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{CE9FE264-6139-47BF-970C-D1BBFCD7EFEA}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{CE9FE264-6139-47BF-970C-D1BBFCD7EFEA}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{CE9FE264-6139-47BF-970C-D1BBFCD7EFEA}.Release|Any CPU.Build.0 = Release|Any CPU
+		{87CFA3EF-2A15-4861-A6DC-C74E9CA3EBCE}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{87CFA3EF-2A15-4861-A6DC-C74E9CA3EBCE}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{87CFA3EF-2A15-4861-A6DC-C74E9CA3EBCE}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{87CFA3EF-2A15-4861-A6DC-C74E9CA3EBCE}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE
@@ -79,6 +92,8 @@ Global
 		{8CC87457-C579-4E39-A1C4-BAADADF500DE} = {245AA4AB-ECF4-4C69-B63D-C59D4B3F918C}
 		{01E5055D-BF6A-435C-8858-F63A8FAC6D86} = {8DE53FA4-6C58-4FE4-840C-8CD5D5370324}
 		{B3DEE02F-9CD3-4009-AFEE-00F0B09811EC} = {245AA4AB-ECF4-4C69-B63D-C59D4B3F918C}
+		{CE9FE264-6139-47BF-970C-D1BBFCD7EFEA} = {245AA4AB-ECF4-4C69-B63D-C59D4B3F918C}
+		{87CFA3EF-2A15-4861-A6DC-C74E9CA3EBCE} = {8DE53FA4-6C58-4FE4-840C-8CD5D5370324}
 	EndGlobalSection
 	GlobalSection(ExtensibilityGlobals) = postSolution
 		SolutionGuid = {823D9407-93C0-443E-89EC-D9DBB006A195}

diff --git a/appveyor.yml b/appveyor.yml
@@ -1,8 +1,8 @@
 version: 1.0.{build}
 skip_tags: true
-stack: node 9
+stack: node 9, redis
 skip_commits:
-  message: /\[chore\]/
+  message: /^chore(release)/
 branches:
   only:
   - master
@@ -26,6 +26,9 @@ install:
     - sh: nuget install GitVersion.CommandLine -ExcludeVersion
     - sh: mono GitVersion.CommandLine/tools/GitVersion.exe /l console /output buildserver
     - ps: if ($isWindows) { .\dotnet-install.ps1 -Version 2.2.100 }
+    - cmd: nuget install redis-64 -excludeversion
+    - cmd: redis-64\tools\redis-server.exe --service-install
+    - cmd: redis-64\tools\redis-server.exe --service-start
     - cmd: gitversion /l console /output buildserver
     - cmd: nuget install ReportGenerator -ExcludeVersion
     - ps: ./appveyorinit.ps1
@@ -49,7 +52,7 @@ deploy:
       auth_token: $(GH_TOKEN)
       draft: true
       prerelease: true
-      release: $(APPVEYOR_BUILD_VERSION)
+      release: $(Version)
       on:
         branch: 
             - /preview*/ 
@@ -83,10 +86,10 @@ for:
       - cmd: git config --global user.name "Aguacongas"
       - cmd: git checkout gh-pages
       - cmd: git stash
-      - cmd: mkdir %APPVEYOR_BUILD_VERSION%
-      - cmd: move coverage\docs %APPVEYOR_BUILD_VERSION%
-      - cmd: git add %APPVEYOR_BUILD_VERSION%
-      - cmd: git commit %APPVEYOR_BUILD_VERSION% -m "Appveyor build succed %APPVEYOR_BUILD_NUMBER%"
+      - cmd: mkdir %Version%
+      - cmd: move coverage\docs %Version%
+      - cmd: git add %Version%
+      - cmd: git commit %Version% -m "Appveyor build succed %APPVEYOR_BUILD_NUMBER%"
       - cmd: git push
 -
   branches:

diff --git a/appveyorinit.ps1 b/appveyorinit.ps1
@@ -14,14 +14,14 @@ else
 }
 
 appveyor SetVariable -Name SemVer -Value $nextversion
+appveyor AddMessage "SemVer = $nextversion"
 
 if (![string]::IsNullOrEmpty($env:GitVersion_PreReleaseLabel))
 {
     $nextversion = "$nextversion-$env:GitVersion_PreReleaseLabel$env:GitVersion_CommitsSinceVersionSourcePadded"
 }
-
+appveyor SetVariable -Name Version -Value $nextversion
 appveyor UpdateBuild -Version $nextversion
-$builnumbersuffix = Get-Date -Format "mmddyyyy-HHmm"
-$builnumber = "$builnumber-$builnumbersuffix"
+appveyor AddMessage "Version = $nextversion"
 
 dotnet restore
diff --git a/sample/Aguacongas.AspNetCore.Authentication.Sample/Controllers/HomeController.cs b/sample/Aguacongas.AspNetCore.Authentication.Sample/Controllers/HomeController.cs
@@ -30,6 +30,7 @@ public IActionResult Index()
             return View(_manager.ManagedHandlerType.Select(t => t.Name));
         }
 
+        // Returns an empty details view to create a scheme for a type of handler
         [Route("Create/{type}")]
         public IActionResult Create(string type)
         {
@@ -39,6 +40,7 @@ public IActionResult Create(string type)
             });
         }
 
+        // Creates a new scheme
         [HttpPost]
         [Route("Create/{type}")]
         public async Task<IActionResult> Create(AuthenticationViewModel model)
@@ -72,25 +74,15 @@ await _manager.AddAsync(new SchemeDefinition
             return View(model);
         }
 
+        // Returns a scheme details view to update id
         [Route("Update/{scheme}")]
         public async Task<IActionResult> Update(string scheme)
         {
             AuthenticationViewModel model;
             var definition = await _manager.FindBySchemeAsync(scheme);
             if (definition == null)
             {
-                var schemes = await _signInManager.GetExternalAuthenticationSchemesAsync();
-                var authenticationScheme = schemes.FirstOrDefault(s => s.Name == scheme);
-                if (authenticationScheme == null)
-                {
-                    return NotFound();
-                }
-                model = new AuthenticationViewModel
-                {
-                    Scheme = authenticationScheme.Name,
-                    DisplayName = authenticationScheme.DisplayName,
-                    HandlerType = authenticationScheme.HandlerType.Name
-                };
+                return NotFound();
             }
             else
             {
@@ -101,20 +93,15 @@ public async Task<IActionResult> Update(string scheme)
                     HandlerType = definition.HandlerType.Name
                 };
 
-                if (definition.Options is OAuthOptions oAuthOptions) // GoogleOptions is OAuthOptions
-                {
-                    model.ClientId = oAuthOptions.ClientId;
-                    model.ClientSecret = oAuthOptions.ClientSecret;
-                }
-                else
-                {
-                    return Error();
-                }
+                var oAuthOptions = definition.Options as OAuthOptions; // GoogleOptions is OAuthOptions
+                model.ClientId = oAuthOptions.ClientId;
+                model.ClientSecret = oAuthOptions.ClientSecret;
             }
 
             return View(model);
         }
 
+        // Updates a scheme
         [HttpPost]
         [Route("Update/{scheme}")]
         public async Task<IActionResult> Update(AuthenticationViewModel model)
@@ -124,15 +111,13 @@ public async Task<IActionResult> Update(AuthenticationViewModel model)
                 var definition = await _manager.FindBySchemeAsync(model.Scheme);
                 if (definition == null)
                 {
-                    await Create(model);
-
-                    return View(model);
+                    return NotFound();
                 }
 
                 if (definition.Options is OAuthOptions oAuthOptions) // GoogleOptions is OAuthOptions
                 {
                     oAuthOptions.ClientId = model.ClientId;
-                    oAuthOptions.ClientSecret = model.ClientSecret;                    
+                    oAuthOptions.ClientSecret = model.ClientSecret;
                 }
 
                 definition.DisplayName = model.DisplayName;
@@ -143,6 +128,7 @@ public async Task<IActionResult> Update(AuthenticationViewModel model)
             return View(model);
         }
 
+        // Lists all schemes we can manage
         [Route("List")]
         public async Task<IActionResult> List()
         {
@@ -160,9 +146,16 @@ public async Task<IActionResult> List()
             }));
         }
 
+        // Deletes a scheme
         [Route("Delete/{scheme}")]
         public async Task<IActionResult> Delete(string scheme)
         {
+            var definition = await _manager.FindBySchemeAsync(scheme);
+            if (definition == null)
+            {
+                return NotFound();
+            }
+
             await _manager.RemoveAsync(scheme);
             return RedirectToAction("List");
         }

diff --git a/...uthentication.EntityFramework/Aguacongas.AspNetCore.Authentication.EntityFramework.csproj b/...uthentication.EntityFramework/Aguacongas.AspNetCore.Authentication.EntityFramework.csproj
@@ -22,6 +22,10 @@
     <DocumentationFile>C:\Projects\Perso\DymamicAuthProviders\src\Aguacongas.AspNetCore.Authentication.EntityFramework\Aguacongas.AspNetCore.Authentication.EntityFramework.xml</DocumentationFile>
   </PropertyGroup>
 
+  <ItemGroup>
+    <None Remove="Aguacongas.AspNetCore.Authentication.EntityFramework.xml" />
+  </ItemGroup>
+
   <ItemGroup>
     <PackageReference Include="Microsoft.EntityFrameworkCore" Version="2.2.0" />
   </ItemGroup>