Skip to content

Conversation

@dkbennett
Copy link
Member

@dkbennett dkbennett commented May 27, 2025

Adds in GPO support ported over from Dev Home.

Notable changes

  • Path of the policy regkey was updated to be HKLM\Software\Policies\Microsoft\Windows\WindowsAdvancedSettings to be consistent with other Windows policies.
  • Logging changed to be a bit less noisy on the happy path of NotConfigured (shows nothing) and will only show errors if an invalid configuration was supplied.
  • Folder configuration moved to the SourceControlIntegration project.
  • Added support to remove all registered folders for a provider.
  • If Advanced Settings is disabled by GPO and is launched, it will also remove all registered folders for Advanced Settings so File Explorer will not try to restart it.
  • Added support for removing all registered folders for a provider to the tester app.

Policy was added to the COM server launches, but not the main app since Settings uses that for configuration data, I did not want to break Settings.

The COM servers for source control integration will not launch if the policy is disabled.

Tested

  • Verified setting the policy prevents COM servers from activating.

It is notable that I observed File Explorer will attempt to restart the COM servers every 30 seconds, but this only happens once since the folder path is de-registered when GPO is disabled. Explorer will launch the COM server one additional time, it will exit, and then not be re-attempted.

@dkbennett dkbennett requested a review from cinnamon-msft May 29, 2025 20:17
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are there two copies of this file?

Licensed under the MIT License. -->
<policyDefinitions xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" revision="0.1900" schemaVersion="1.0" xmlns="http://schemas.microsoft.com/GroupPolicy/2006/07/PolicyDefinitions">
<policyNamespaces>
<target prefix="devhome" namespace="Microsoft.Policies.DevHome" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still using the DevHome name here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants