microsoft / vscode-remote-release

Visual Studio Code Remote Development: Open any folder in WSL, in a Docker container, or on a remote machine using SSH and take advantage of VS Code's full feature set.
https://aka.ms/vscode-remote
Other
3.64k stars 285 forks source link

Remote Extension requires Microsoft account #9615

Open cemde opened 7 months ago

cemde commented 7 months ago

Steps to Reproduce:

  1. Fresh install of VSCode and remote extension

Does this issue occur when you try this locally?: Yes Does this issue occur when you try this locally and all extensions are disabled?: No


The Avatar in the bottom left corner shows a notification symbol for a notification Sign in with Microsoft to use Remote - Tunnels (1). When run locally, the notification only starts appearing after clicking on the remote extension symbol in the menu side bar. I do not use Remote - Tunnels and do not have a Microsoft account I want to associate with VSCode. The notification is distracting. I assume this is an error.

I tried reinstalling the extensions, which didn't work.

cwwalter commented 7 months ago

A bit more probably related info: I am seeing this too. I do use tunnels but I only am using them through GitHub.

I don't think I saw this before but I am seeing it now. I see both the notification in the lower left as noted above by @cemde and I am also seeing it here:

image

where both of those tunnels only use GitHub.

cwwalter commented 7 months ago

Note the messages to disappear if you log into a Microsoft account. From previous work related to proxies and firewalls I know that even if you are using GitHub only for your tunnels it is contacting azure. So, at a minimum the text is misleading since in my case it is apparently necessary although I have no tunnels registered with Microsoft.

cemde commented 7 months ago
Screenshot 2024-03-16 at 22 45 01

Here is a screenshot of what I see

cemde commented 6 months ago

@chrmarti Are there plans to fix this bug?

connor4312 commented 6 months ago

@TylerLeonhardt we probe for accounts on both providers but don't want to show a notification here, is there a way to avoid it?

TylerLeonhardt commented 6 months ago

@connor4312 silent: true property on getSession

cemde commented 3 months ago

@connor4312 any updates? :)

bug still exists after today's update.