Merge pull request #125 from microsoft/master

Cut another release to fix the bug in 2.0.153

Author: mjcheetham

.github/workflows/release-homebrew.yaml

@@ -11,6 +11,7 @@ jobs:
       uses: actions/[email protected]
       id: version
       with:
+        github-token: ${{secrets.GITHUB_TOKEN}}
         result-encoding: string
         script: |
           const { data } = await github.repos.getReleaseByTag({

src/shared/Microsoft.Git.CredentialManager/Interop/Windows/WindowsSessionManager.cs

@@ -20,7 +20,7 @@ public unsafe bool IsDesktopSession
                 // In .NET 5 the implementation on Windows has been 'fixed', but still POSIX versions always return true.
                 //
                 // This code is lifted from the .NET 5 targeting dotnet/runtime implementation for Windows:
-                // https://github.com/dotnet/runtime/blob/8c10a98c13263eea806d4fcecb76d29e38593539/src/libraries/System.Private.CoreLib/src/System/Environment.Windows.cs#L125-L145
+                // https://github.com/dotnet/runtime/blob/cf654f08fb0078a96a4e414a0d2eab5e6c069387/src/libraries/System.Private.CoreLib/src/System/Environment.Windows.cs#L125-L145
 
                 // Per documentation of GetProcessWindowStation, this handle should not be closed
                 IntPtr handle = User32.GetProcessWindowStation();
@@ -31,7 +31,7 @@ public unsafe bool IsDesktopSession
                     if (User32.GetUserObjectInformation(handle, User32.UOI_FLAGS, &flags,
                         (uint) sizeof(USEROBJECTFLAGS), ref dummy))
                     {
-                        return (flags.dwFlags & User32.WSF_VISIBLE) == 0;
+                        return (flags.dwFlags & User32.WSF_VISIBLE) != 0;
                     }
                 }