Merge pull request #61 from RoushTech/release/0.6.3

Open 0.6.3, remove key from source.
RoushTech · Sep 4, 2017 · 1622195 · 1622195
2 parents 79135be + 95d9c75
commit 1622195
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,9 @@
-# 0.6.2
+# 0.6.3
+<sup>Released: 2017/09/04</sup>
+
+- Use per-request dependencies for handling RollbarClient to prevent errors.
+
+# 0.6.2
 <sup>Released: 2017/8/20</sup>
 
 - Change method trace string to have full namespace, method name and parameters.

diff --git a/src/RollbarDotNet/Core/ExceptionHandlerMiddleware.cs b/src/RollbarDotNet/Core/ExceptionHandlerMiddleware.cs
@@ -7,26 +7,22 @@
     public class ExceptionHandlerMiddleware
     {
         public ExceptionHandlerMiddleware(
-            RequestDelegate next,
-            Rollbar rollbar)
+            RequestDelegate next)
         {
             this.Next = next;
-            this.Rollbar = rollbar;
         }
 
         protected RequestDelegate Next { get; set; }
-
-        protected Rollbar Rollbar { get; set; }
 
-        public async Task Invoke(HttpContext context)
+        public async Task Invoke(HttpContext context, Rollbar rollbar)
         {
             try
             {
                 await this.Next(context);
             }
             catch(Exception exception)
             {
-                var response = await this.Rollbar.SendException(exception);
+                var response = await rollbar.SendException(exception);
                 var rollbarResponseFeature = new RollbarResponseFeature
                 {
                     Handled = true,

diff --git a/src/RollbarDotNet/RollbarDotNet.csproj b/src/RollbarDotNet/RollbarDotNet.csproj
@@ -4,7 +4,7 @@
     <Description>Rollbar Integration for .NET Core</Description>
     <Copyright>William Roush 2016-2017</Copyright>
     <AssemblyTitle>RollbarDotNet</AssemblyTitle>
-    <VersionPrefix>0.6.2</VersionPrefix>
+    <VersionPrefix>0.6.3</VersionPrefix>
     <Authors>William Roush</Authors>
     <AssemblyName>RollbarDotNet</AssemblyName>
     <PackageId>RollbarDotNet</PackageId>

diff --git a/test/RollbarDotNet.Tests/DependencyInjection/WebDependencyInjectionTests.cs b/test/RollbarDotNet.Tests/DependencyInjection/WebDependencyInjectionTests.cs
@@ -21,7 +21,7 @@ public WebDependencyInjectionTests()
                     .AddSingleton(mockHostingEnvironment.Object);
             services.Configure<RollbarOptions>(o =>
             {
-                o.AccessToken = Environment.GetEnvironmentVariable("ROLLBAR_TOKEN") ?? "992eb96a7d4b4d62bd529188bb8152cb";
+                o.AccessToken = Environment.GetEnvironmentVariable("ROLLBAR_TOKEN");
                 o.Environment = "Testing";
             });