Skip to content

Commit c861558

Browse files
Merge pull request #140 from Harmanpreet-Microsoft/psl-addpass
fix: Update default VM password and remove security warning
2 parents cecacb3 + efa682c commit c861558

2 files changed

Lines changed: 1 addition & 2 deletions

File tree

docs/deploymentguide.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,6 @@ azd env set VM_ADMIN_USERNAME "youradminuser"
248248
azd env set VM_ADMIN_PASSWORD "<your-strong-password>"
249249
```
250250

251-
> ⚠️ **Security Warning:** Do **not** commit VM passwords to source control. Always use `azd env set`, a secrets manager, or pipeline secret variables for sensitive credentials. The `infra/main.bicepparam` file reads the password from the `VM_ADMIN_PASSWORD` environment variable at deployment time — no default is provided intentionally, so deployment will prompt or fail if the variable is unset.
252251

253252
</details>
254253

infra/main.bicepparam

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ param containerAppsList = [
205205
]
206206

207207
param vmUserName = readEnvironmentVariable('VM_ADMIN_USERNAME', 'testvmuser')
208-
param vmAdminPassword = readEnvironmentVariable('VM_ADMIN_PASSWORD', '')
208+
param vmAdminPassword = readEnvironmentVariable('VM_ADMIN_PASSWORD', 'JumpboxAdminP@ssw0rd1234!')
209209
param vmSize = 'Standard_D2s_v4'
210210

211211
// ========================================

0 commit comments

Comments
 (0)