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

fixed Shell Launcher Powershell script on configuration/shell-launcher/shell-launcher/index.md #11974

Open
wants to merge 2 commits into
base: public
Choose a base branch
from

Conversation

GiamBoscaro
Copy link

@GiamBoscaro GiamBoscaro commented Dec 12, 2024

Description

The Powershell script shown in the docs to apply the Shell Launcher configuration is not working. The configuration cannot be applied, and if error happens the script cannot retrieve the errors from the Windows Event Viewer.

Why

Why? I am not a Windows expert at all and I have wasted days trying to setup the Shell Launcher. I have finally made it and I have figured that the documentation was actually wrong in multiple ways. I would like to update the docs to avoid this kind of trouble for other people.

Changes

  • The XML configuration must be set into the ShellLauncher property and not Configuration property of the CimInstance.
  • $eventLogFilterHashTable was undefined and the script would break if some errors were to happen while applying the Shell Launcher configuration.

Copy link
Contributor

Learn Build status updates of commit 03d522d:

✅ Validation status: passed

File Status Preview URL Details
windows/configuration/assigned-access/shell-launcher/index.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 66e02ec:

✅ Validation status: passed

File Status Preview URL Details
windows/configuration/assigned-access/shell-launcher/index.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

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.

1 participant