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
core - don't expand {now} placeholder during provisioning #8509
core - don't expand {now} placeholder during provisioning #8509
Conversation
7a33b8f
to
e486aa0
Compare
b7fc104
to
28763d4
Compare
we should probably have a test here wrt to the current work around re double escape Ala {{now}} as we to validate not breaking extant behavior. |
Fair point, added that case. |
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.
thanks, lgtm
I think there's at least one other issue that gets closed from this as well.
Today we expand the
{now}
placeholder when a policy executes, regardless of whether we're provisioning a policy or actually running it. This change lets us defer expansion while preserving custom format specifiers.There may be a more straightforward way to do this...
Closes #8503