Fix issue with whitespace in ADML data #55795
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Fixes an issue with the text field in ADML data containing an extra space at the end.
This happens with the Legacy Templates from Microsoft. The
KeepAliveTime
value of300000 or 5 minutes (recommended)
has a space at the end. This causes states to report that a change is needed when in fact it the policy is already set.When the state does the comparison it is comparing the requested dict:
with the current setting
What issues does this PR fix or reference?
Found in Enterprise
Previous Behavior
test=True
will always report a change is necessary.New Behavior
test=True
reports no changes neededTests written?
Yes
Commits signed with GPG?
Yes