-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
win reg - Support special chars in path #53305
Conversation
After speaking with @nitzmahone we've agreed that the changes here are too big and are slightly a behavioral change. Because of this, it won't be backported to earlier releases but will be available in Ansible 2.8. The latest commit adds a deprecation warning when |
"HKCC:" { [Microsoft.Win32.Registry]::CurrentConfig } | ||
"HKCU:" { [Microsoft.Win32.Registry]::CurrentUser } | ||
"HKLM:" { [Microsoft.Win32.Registry]::LocalMachine } | ||
"HKU" { [Microsoft.Win32.Registry]::Users } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe you're missing the colon after HKU here...
SUMMARY
The main part of this module is to support using special characters like
/
in a registry path. Previously/
would be considered a path separator causing it to create a key within a key. The PR also changeswin_reg_stat
to work with the registry key's(Default)
value. This is returned under the""
key if noname
is set but can also be queried directly withname: ''
. The property is considered to exist if a value has been set.This first part includes a fix for
win_regedit
andwin_reg_stat
that is backportable to older branches. A future PR will include a deprecation warning for paths that currently use/
as a path separator.Fixes #41791.
ISSUE TYPE
COMPONENT NAME
win_regedit
win_reg_stat