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

Delay binding for and template function #1026

Closed
BernieWhite opened this issue Oct 11, 2021 · 0 comments · Fixed by #1028
Closed

Delay binding for and template function #1026

BernieWhite opened this issue Oct 11, 2021 · 0 comments · Fixed by #1028
Assignees
Labels
bug Something isn't working
Milestone

Comments

@BernieWhite
Copy link
Collaborator

Description of the issue

Consider [and(not(empty(parameters('blobContainers'))),contains(parameters('blobContainers')[0],'enableWORM'),parameters('blobContainers')[0].enableWORM)]

When this template function is used prior to resolving and the expression parameters('blobContainers')[0].enableWORM is evaluated. However when enableWORM is not a property of the object an error will occur.

The logical and however should never need to evaluate the next parameter if a previous parameter already return false.

Expected behaviour

The and function should delay resolving parameters until it needs them.

Module in use and version:

  • Module: PSRule.Rules.Azure
  • Version: 1.9.0-B2110025

Captured output from $PSVersionTable:

Name                           Value
----                           -----
PSVersion                      7.1.4
PSEdition                      Core
GitCommitId                    7.1.4
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0
@BernieWhite BernieWhite added the bug Something isn't working label Oct 11, 2021
@BernieWhite BernieWhite added this to the v1.9.0 milestone Oct 11, 2021
@BernieWhite BernieWhite self-assigned this Oct 11, 2021
BernieWhite added a commit to BernieWhite/PSRule.Rules.Azure that referenced this issue Oct 11, 2021
BernieWhite added a commit that referenced this issue Oct 11, 2021
…1028)

* Fixes template functions and KeyVault rule #1024 #1025 #1026 #1027

* Improvements to reason message name

* Fix usage of ubuntu-16.04 for container hosting
@BernieWhite BernieWhite mentioned this issue Oct 25, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant