Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows: Investigate changes to API behavior for Wi-Fi access and location #196746

Closed
joaomoreno opened this issue Oct 26, 2023 · 6 comments
Closed
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded windows VS Code on Windows issues

Comments

@joaomoreno
Copy link
Member

Due to this upcoming breaking change in Windows, it seems like VS Code is shown as an application which occasionally asks for Location:

We are seeing location access prompts being generated by this app due to a breaking change in Windows 2024 and want to be sure that this is expected, and that no fixes in the app need to be made.
Changes to API behavior for Wi-Fi access and location - Win32 apps | Microsoft Learn

Is VS Code calling either the Wi-Fi or Location APIs and has a use case for location? If not, this may result in end-user confusion.

As for rollout schedule:

  • In a Windows Insider Preview release, the changes will first take effect in October 2023.
  • You'll then have until the following Windows feature release in fall of 2024 to test your app's scenarios, and to make any necessary changes. After that date, Windows devices will begin getting the update that contains the behavior change described above.
@joaomoreno joaomoreno added bug Issue identified by VS Code Team member as probable bug windows VS Code on Windows issues labels Oct 26, 2023
@joaomoreno joaomoreno added this to the Backlog milestone Oct 26, 2023
@deepak1556 deepak1556 modified the milestones: Backlog, December 2023 Dec 5, 2023
@deepak1556
Copy link
Collaborator

Prompt confirmed, affects all Electron applications.

location_prompt

@deepak1556 deepak1556 added upstream Issue identified as 'upstream' component related (exists outside of VS Code) electron Issues and items related to Electron labels Dec 6, 2023
@deepak1556
Copy link
Collaborator

Got steps to retrigger the prompt for further analysis. One thing I found today, prompt does not trigger with Electron 27 based insiders. This hints positive, we might not need to make any changes in the runtime. I will confirm the details in following days.

@deepak1556
Copy link
Collaborator

WlanQueryInterface was being called on application startup in stable as part of creating the network context as seen in the screenshot below
wlan_usage_electron

The relevant code has been refactored and removed from the startup path since this CL https://chromium-review.googlesource.com/c/chromium/src/+/4485483 which first landed in Chromium 115, also validated there are no other usages of the affected api in the latest insiders based on Chromium 118. Closing this issue are resolved, next stable release 1.86 should have the changes as well.

@andreamah andreamah added the verification-steps-needed Steps to verify are needed for verification label Jan 25, 2024
@andreamah
Copy link
Contributor

@deepak1556 @joaomoreno How can we verify this?

@deepak1556
Copy link
Collaborator

Steps for verification:

@deepak1556
Copy link
Collaborator

The bug was about verifying the issue, which I have already done so feel free to skip verifying again.

@deepak1556 deepak1556 added verified Verification succeeded and removed verification-steps-needed Steps to verify are needed for verification labels Jan 26, 2024
@aiday-mar aiday-mar added this to the December / January 2024 milestone Feb 6, 2024
@microsoft microsoft locked and limited conversation to collaborators Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded windows VS Code on Windows issues
Projects
None yet
Development

No branches or pull requests

4 participants