Azure: remove all credentials instead of just the first one

microsoft · Mar 2, 2020 · daf32ee · daf32ee
1 parent 8eaeff4
commit daf32ee
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/src/cascadia/TerminalConnection/AzureConnection.cpp b/src/cascadia/TerminalConnection/AzureConnection.cpp
@@ -956,17 +956,15 @@ namespace winrt::Microsoft::Terminal::TerminalConnection::implementation
             _WriteStringWithNewline(RS_(L"AzureNoTokens"));
             return;
         }
-        while (credList.Size() > 0)
+
+        for (const auto& cred : credList)
         {
             try
             {
-                vault.Remove(credList.GetAt(0));
-            }
-            catch (...)
-            {
-                _WriteStringWithNewline(RS_(L"AzureTokensRemoved"));
-                return;
+                vault.Remove(cred);
             }
+            CATCH_LOG();
         }
+        _WriteStringWithNewline(RS_(L"AzureTokensRemoved"));
     }
 }