[AVM Module Issue]: connectionStrings object value empty after deployments

[riosengineer]

Check for previous/existing GitHub issues

• I have checked for previous/existing GitHub issues

Issue Type?

Bug

Module Name

avm/res/web/site

(Optional) Module Version

0.12.1

Description

Hi team,

I am unsure on why this is happening, but when you want to deploy connectionStrings via the web/sites/config via the AVM module web/site:0.12.1 (latest) it happily deploys the connection string name, type, but does not populate the value. I couldn't find any examples of this using AVM as the e2e test file does not seem to have it (from what I could see when searching): https://github.com/Azure/bicep-registry-modules/blob/main/avm/res/web/site/tests/e2e/webConfig/main.test.bicep or https://github.com/Azure/bicep-registry-modules/blob/main/avm/res/web/site/tests/e2e/webApp.max/main.test.bicep

I've reviewed the REST API and it doesn't seem like there's much to miss, besides the name, value and type for the siteConfig object:

https://learn.microsoft.com/en-us/azure/templates/microsoft.web/sites/config/connectionstrings?pivots=deployment-language-bicep and https://learn.microsoft.com/en-us/azure/templates/microsoft.web/sites/config?pivots=deployment-language-bicep#microsoftwebsitesconfig-objects

Example:

module appService 'br/public:avm/res/web/site:0.12.1' = {
  name: '${uniqueString(deployment().name, location)}-web'
  scope: resourceGroup(rgName)
.....
siteConfig: {
...
  connectionStrings: [
          {
            name: 'DefaultConnection'
            value: 'TestValue' 
            type: 'SQLAzure'
          }
        ]

Of course, when I set it manually, all OK. Also it doesn't matter much what value I use, or if I use a custom type. Same issue.

(Optional) Correlation Id

8802f95a-fdef-4910-b371-433bb8d06d89

[microsoft-github-policy-service]

Important

The "Needs: Triage 🔍" label must be removed once the triage process is complete!

Tip

For additional guidance on how to triage this issue/PR, see the BRM Issue Triage documentation.

[avm-team-linter]

@riosengineer, thanks for submitting this issue for the avm/res/web/site module!

Important

A member of the @Azure/avm-res-web-site-module-owners-bicep or @Azure/avm-res-web-site-module-contributors-bicep team will review it soon!

[tsc-buddy]

Hey @riosengineer -thanks for opening an issue. We shall schedule some time in Jan to triage this further.

Side note - the site/config--web is going to have some minor changes shortly so we will look at what you mentioned at the same time.

[riosengineer]

Hey @riosengineer -thanks for opening an issue. We shall schedule some time in Jan to triage this further.

Side note - the site/config--web is going to have some minor changes shortly so we will look at what you mentioned at the same time.

Thanks buddy, look forward to the changes. Shout if you need any more info or testing!

[microsoft-github-policy-service]

Warning

Tagging the AVM Core Team (@Azure/avm-core-team-technical-bicep) due to a module owner or contributor having not responded to this issue within 3 business days. The AVM Core Team will attempt to contact the module owners/contributors directly.

Tip

• To prevent further actions to take effect, the "Status: Response Overdue 🚩" label must be removed, once this issue has been responded to.
• To avoid this rule being (re)triggered, the ""Needs: Triage 🔍" label must be removed as part of the triage process (when the issue is first responded to)!