Skip to content

Commit

Permalink
Try fix some bicep errors
Browse files Browse the repository at this point in the history
  • Loading branch information
JakeGinnivan committed Jul 18, 2024
1 parent eeb6c17 commit aa4e8b3
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
5 changes: 4 additions & 1 deletion infra/app/ddd-2024.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ param name string
param location string = resourceGroup().location
param tags object = {}

param environment string
param authClientId string
param authClientSecret string
param identityName string
param containerRegistryName string
param containerAppsEnvironmentName string
Expand Down Expand Up @@ -128,7 +131,7 @@ output id string = app.id

resource authConfig 'Microsoft.App/containerApps/authConfigs@2023-11-02-preview' = if (environment == 'dev') {
name: 'authconfig'
parent: containerAppResource
parent: app
properties: {
globalValidation: {
unauthenticatedClientAction: 'RedirectToLoginPage'
Expand Down
9 changes: 8 additions & 1 deletion infra/main.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,15 @@ param ddd2024Exists bool
@secure()
param ddd2024Definition object

param authClientId string
@secure()
param authClientSecret string

@description('Id of the user or app to assign application roles')
param principalId string

// Tags that should be applied to all resources.
//
//
// Note that 'azd-service-name' tags should be applied separately to service host resources.
// Example usage:
// tags: union(tags, { 'azd-service-name': <service name in azure.yaml> })
Expand Down Expand Up @@ -101,6 +105,9 @@ module ddd2024 './app/ddd-2024.bicep' = {
containerRegistryName: registry.outputs.name
exists: ddd2024Exists
appDefinition: ddd2024Definition
environment: environmentName
authClientId: authClientId
authClientSecret: authClientSecret
}
scope: rg
}
Expand Down

0 comments on commit aa4e8b3

Please sign in to comment.