Merge pull request #162 from deisterhold/patch-1

Tiago Brenck · web-flow · commit 7a67374ddcc2 · 2019-08-14T11:01:00.000-07:00
Fix cache options being ignored
diff --git a/Microsoft.Identity.Web/Client/TokenCacheProviders/InMemory/MSALAppMemoryTokenCacheProvider.cs b/Microsoft.Identity.Web/Client/TokenCacheProviders/InMemory/MSALAppMemoryTokenCacheProvider.cs
@@ -60,7 +60,7 @@ public MSALAppMemoryTokenCacheProvider(IMemoryCache cache,
             MSALMemoryTokenCacheOptions option,
             IOptionsMonitor<AzureADOptions> azureAdOptionsAccessor)
         {
-            if (option != null)
+            if (option == null)
             {
                 this.CacheOptions = new MSALMemoryTokenCacheOptions();
             }
@@ -132,4 +132,4 @@ private void AppTokenCacheAfterAccessNotification(TokenCacheNotificationArgs arg
             }
         }
     }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ public MSALAppMemoryTokenCacheProvider(IMemoryCache cache,`
`60`	`60`	`MSALMemoryTokenCacheOptions option,`
`61`	`61`	`IOptionsMonitor<AzureADOptions> azureAdOptionsAccessor)`
`62`	`62`	`{`
`63`		`- if (option != null)`
	`63`	`+ if (option == null)`
`64`	`64`	`{`
`65`	`65`	`this.CacheOptions = new MSALMemoryTokenCacheOptions();`
`66`	`66`	`}`
`@@ -132,4 +132,4 @@ private void AppTokenCacheAfterAccessNotification(TokenCacheNotificationArgs arg`
`132`	`132`	`}`
`133`	`133`	`}`
`134`	`134`	`}`
`135`		`-}`
	`135`	`+}`