Docker image for windows is now complete.

Author: jgauffin

ReadMe.md

@@ -30,7 +30,10 @@ With Coderr, you can focus on building new features and spend minimal time corre
 
 ## Getting started
 
-Once you have [downloaded and installed the server](https://github.com/coderrio/Coderr.Server/releases) you need to install and configure one of our nuget packages. You can read more about them [here](https://coderr.io/documentation/client/).
+1. [Download Coderr Server](https://github.com/coderrio/Coderr.Server/releases), use our [cloud service](https://app.coderr.io) (free for up to five users) or use our [Docker image]()
+2. Install one of our [nuget libraries](https://www.nuget.org/packages?q=coderr.client) (or [npm library](https://www.npmjs.com/package/coderr.client)).
+3. Follow the instructions in the package ReadMe (max three lines of code to get started).
+4. Try the code below.
 
 **Unhandled exceptions will automatically be reported by the client libraries.**
 

src/DockerCompose/Windows/docker-compose.yml

@@ -19,10 +19,11 @@ services:
 
       # Comment out this line once Coderr have been configured (run through the install wizard)
       # and then restart the container.
-      # - CODERR_CONFIG_PASSWORD=changeThis2
+      - CODERR_CONFIG_PASSWORD=changeThis
+      
+    volumes:
+      - ${APPDATA}/Microsoft/UserSecrets:C:\ProtectedStorage:rw
+
     ports:
       - "60473:5000/tcp"
       - "60474:5001"
-      
-      
-      

src/Server/Coderr.Server.ReportAnalyzer.Abstractions/ErrorReports/ReportDTO.cs

@@ -7,6 +7,16 @@ namespace Coderr.Server.ReportAnalyzer.Abstractions.ErrorReports
     /// </summary>
     public class ReportDTO
     {
+        public ReportDTO(int id)
+        {
+            Id = id;
+        }
+
+        protected ReportDTO()
+        {
+
+        }
+
         /// <summary>
         ///     Application that the incident and report belongs in.
         /// </summary>
@@ -30,7 +40,7 @@ public class ReportDTO
         /// <summary>
         ///     DB primary key
         /// </summary>
-        public int Id { get; set; }
+        public int Id { get; private set; }
 
         /// <summary>
         ///     DB primary key

src/Server/Coderr.Server.ReportAnalyzer.Tests/ErrorOrigins/Handlers/StorePositionFromNewReportTests.cs

@@ -22,7 +22,7 @@ public async Task Should_be_able_to_use_position_in_CoderrCollection()
             var repos = Substitute.For<IErrorOriginRepository>();
             var incident = new IncidentSummaryDTO(1, "Hello");
             var data = new Dictionary<string, string> {{"Longitude", "60.6065"}, {"Latitude", "15.6355"}};
-            var report = new ReportDTO {ContextCollections = new[] {new ContextCollectionDTO("CoderrData", data)}};
+            var report = new ReportDTO(1) {ContextCollections = new[] {new ContextCollectionDTO("CoderrData", data)}};
             var e = new ReportAddedToIncident(incident, report, false);
             var context = Substitute.For<IMessageContext>();
             var configWrapper = Substitute.For<IConfiguration<OriginsConfiguration>>();
@@ -42,7 +42,7 @@ public async Task Should_be_able_to_use_position_in_regular_collection()
             var repos = Substitute.For<IErrorOriginRepository>();
             var incident = new IncidentSummaryDTO(1, "Hello");
             var data = new Dictionary<string, string> {{"ReportLongitude", "60.6065"}, {"ReportLatitude", "15.6355"}};
-            var report = new ReportDTO {ContextCollections = new[] {new ContextCollectionDTO("SomeCollection", data)}};
+            var report = new ReportDTO(1) {ContextCollections = new[] {new ContextCollectionDTO("SomeCollection", data)}};
             var e = new ReportAddedToIncident(incident, report, false);
             var context = Substitute.For<IMessageContext>();
             var configWrapper = Substitute.For<IConfiguration<OriginsConfiguration>>();

src/Server/Coderr.Server.ReportAnalyzer.Tests/Tagging/IdentifyTagsFromIncidentTests.cs

@@ -23,7 +23,7 @@ public async Task should_be_able_to_identity_tags_when_only_one_is_specified()
             provider.GetIdentifiers(Arg.Any<TagIdentifierContext>()).Returns(new ITagIdentifier[0]);
             var ctx = Substitute.For<IMessageContext>();
             var incident = new IncidentSummaryDTO(1, "Ada");
-            var report = new ReportDTO
+            var report = new ReportDTO(1)
             {
                 ContextCollections = new[]
                     {new ContextCollectionDTO("Data", new Dictionary<string, string> {{"ErrTags", "MyTag"}})}
@@ -47,7 +47,7 @@ public async Task should_be_able_to_identity_tags_when_only_multiple_tags_are_sp
             provider.GetIdentifiers(Arg.Any<TagIdentifierContext>()).Returns(new ITagIdentifier[0]);
             var ctx = Substitute.For<IMessageContext>();
             var incident = new IncidentSummaryDTO(1, "Ada");
-            var report = new ReportDTO
+            var report = new ReportDTO(1)
             {
                 ContextCollections = new[]
                     {new ContextCollectionDTO("Data", new Dictionary<string, string> {{"ErrTags", "MyTag,YourTag"}})}
@@ -63,30 +63,6 @@ public async Task should_be_able_to_identity_tags_when_only_multiple_tags_are_sp
             tags[0].Name.Should().Be("MyTag");
             tags[1].Name.Should().Be("YourTag");
         }
-
-        [Fact]
-        public async Task should_be_able_to_identity_tags_when_only_multiple_tags_are_specified_with_spaces()
-        {
-            var repos = Substitute.For<ITagsRepository>();
-            var provider = Substitute.For<ITagIdentifierProvider>();
-            provider.GetIdentifiers(Arg.Any<TagIdentifierContext>()).Returns(new ITagIdentifier[0]);
-            var ctx = Substitute.For<IMessageContext>();
-            var incident = new IncidentSummaryDTO(1, "Ada");
-            var report = new ReportDTO
-            {
-                ContextCollections = new[]
-                    {new ContextCollectionDTO("Data", new Dictionary<string, string> {{"ErrTags[]", "MyTag"}, {"ErrTags[]", "YourTag"}})}
-            };
-            var e = new ReportAddedToIncident(incident, report, false);
-
-            var sut = new IdentifyTagsFromIncident(repos, provider);
-            await sut.HandleAsync(ctx, e);
-
-            var arguments = repos.ReceivedCalls().First(x => x.GetMethodInfo().Name == "AddAsync")
-                .GetArguments();
-            var tags = (Tag[]) arguments[1];
-            tags[0].Name.Should().Be("MyTag");
-            tags[1].Name.Should().Be("YourTag");
-        }
+       
     }
 }

src/Server/Coderr.Server.ReportAnalyzer/Inbound/Handlers/Reports/ReportAnalyzer.cs

@@ -314,13 +314,12 @@ private ReportExeptionDTO ConvertToCoreException(ErrorReportException exception)
 
         private ReportDTO ConvertToCoreReport(ErrorReportEntity report, string version)
         {
-            var dto = new ReportDTO
+            var dto = new ReportDTO(report.Id)
             {
                 ApplicationId = report.ApplicationId,
                 ContextCollections =
                     report.ContextCollections.Select(x => new ContextCollectionDTO(x.Name, x.Properties)).ToArray(),
                 CreatedAtUtc = report.CreatedAtUtc,
-                Id = report.Id,
                 IncidentId = report.IncidentId,
                 RemoteAddress = report.RemoteAddress,
                 ReportId = report.ClientReportId,

src/Server/Coderr.Server.Web/Startup.cs

@@ -29,6 +29,7 @@
 using Microsoft.AspNetCore.Authentication;
 using Microsoft.AspNetCore.Authentication.Cookies;
 using Microsoft.AspNetCore.Builder;
+using Microsoft.AspNetCore.DataProtection;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Http.Extensions;
@@ -203,6 +204,11 @@ public void ConfigureServices(IServiceCollection services)
                     jsonOptions.SerializerSettings.ContractResolver = new IncludeNonPublicMembersContractResolver();
                 });
 
+            if (HostConfig.Instance.IsRunningInDocker)
+            {
+                services.AddDataProtection()
+                    .PersistKeysToFileSystem(new System.IO.DirectoryInfo(@"C:\ProtectedStorage"));
+            }
 
             var authenticationBuilder = services.AddAuthentication("Cookies");
             authenticationBuilder.AddCookie(CookieAuthenticationDefaults.AuthenticationScheme, options =>