From 71a98eacf43754a0f041265fec1f78878aca99d4 Mon Sep 17 00:00:00 2001
From: Christopher Suh <chsuh@microsoft.com>
Date: Fri, 30 Oct 2020 13:25:07 -0400
Subject: [PATCH] Add handling of user.UniqueId if not set (#1109)

* commented subscription caching

* added additional code path if user.UniqueId not set

* fixed failing unit tests
---
 .../AzureAuthenticationManager.cs                          | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureAuthenticationManager.cs b/src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureAuthenticationManager.cs
index 43a07b05..4a8b9a08 100644
--- a/src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureAuthenticationManager.cs
+++ b/src/Microsoft.SqlTools.ResourceProvider.DefaultImpl/AzureAuthenticationManager.cs
@@ -222,9 +222,10 @@ namespace Microsoft.SqlTools.ResourceProvider.DefaultImpl
             
             if (user != null)
             {
-                result = _subscriptionCache.Get(user.UniqueId);
-                if (result == null)
-                {
+                if (user.UniqueId != "") {
+                    result = _subscriptionCache.Get(user.UniqueId);
+                }
+                if (result == null) {
                     result = await GetSubscriptionFromServiceAsync(user);
                     _subscriptionCache.UpdateCache(user.UniqueId, result);
                 }