Merge pull request #550 from stelucz/fix-pass-auth

tanmay-hc · web-flow · commit b321cdd9caf3 · 2025-12-15T12:22:53.000+05:30
Fix disablePasswordAuthentication option
diff --git a/builder/azure/arm/template_factory_test.TestBasicSkuPublicIPVMDeployment.approved.json b/builder/azure/arm/template_factory_test.TestBasicSkuPublicIPVMDeployment.approved.json
@@ -133,6 +133,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestConfidentialVM01.approved.json b/builder/azure/arm/template_factory_test.TestConfidentialVM01.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestConfidentialVM02.approved.json b/builder/azure/arm/template_factory_test.TestConfidentialVM02.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestConfidentialVM03.approved.json b/builder/azure/arm/template_factory_test.TestConfidentialVM03.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestEncryptionAtHost01.approved.json b/builder/azure/arm/template_factory_test.TestEncryptionAtHost01.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestEncryptionAtHost02.approved.json b/builder/azure/arm/template_factory_test.TestEncryptionAtHost02.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestSigSourcedWithDiskEncryptionSet.approved.json b/builder/azure/arm/template_factory_test.TestSigSourcedWithDiskEncryptionSet.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestTrustedLaunch01.approved.json b/builder/azure/arm/template_factory_test.TestTrustedLaunch01.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment03.approved.json b/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment03.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment05.approved.json b/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment05.approved.json
@@ -94,6 +94,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment07.approved.json b/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment07.approved.json
@@ -117,6 +117,7 @@
           "computerName": "[parameters('vmName')]",
           "customData": "I2Nsb3VkLWNvbmZpZwpncm93cGFydDoKICBtb2RlOiBvZmYK",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment09.approved.json b/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment09.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment11.approved.json b/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment11.approved.json
@@ -116,6 +116,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment15.approved.json b/builder/azure/arm/template_factory_test.TestVirtualMachineDeployment15.approved.json
@@ -120,6 +120,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/common/template/template_builder.go b/builder/azure/common/template/template_builder.go
@@ -70,6 +70,8 @@ func (s *TemplateBuilder) BuildLinux(sshAuthorizedKey string, disablePasswordAut
 	if disablePasswordAuthentication {
 		profile.LinuxConfiguration.DisablePasswordAuthentication = common.BoolPtr(true)
 		profile.AdminPassword = nil
+	} else {
+		profile.LinuxConfiguration.DisablePasswordAuthentication = common.BoolPtr(false)
 	}
 
 	s.osType = hashiVMSDK.OperatingSystemTypesLinux
diff --git a/builder/azure/common/template/template_builder_test.TestBuildLinux01.approved.json b/builder/azure/common/template/template_builder_test.TestBuildLinux01.approved.json
@@ -133,6 +133,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/common/template/template_builder_test.TestCommunitySharedImageGallery00.approved.json b/builder/azure/common/template/template_builder_test.TestCommunitySharedImageGallery00.approved.json
@@ -133,6 +133,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/common/template/template_builder_test.TestDirectSharedImageGallery00.approved.json b/builder/azure/common/template/template_builder_test.TestDirectSharedImageGallery00.approved.json
@@ -133,6 +133,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/common/template/template_builder_test.TestNetworkSecurityGroup00.approved.json b/builder/azure/common/template/template_builder_test.TestNetworkSecurityGroup00.approved.json
@@ -112,6 +112,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {
diff --git a/builder/azure/common/template/template_builder_test.TestSharedImageGallery00.approved.json b/builder/azure/common/template/template_builder_test.TestSharedImageGallery00.approved.json
@@ -133,6 +133,7 @@
           "adminUsername": "[parameters('adminUsername')]",
           "computerName": "[parameters('vmName')]",
           "linuxConfiguration": {
+            "disablePasswordAuthentication": false,
             "ssh": {
               "publicKeys": [
                 {

Original file line number	Diff line number	Diff line change
`@@ -133,6 +133,7 @@`
`133`	`133`	`"adminUsername": "[parameters('adminUsername')]",`
`134`	`134`	`"computerName": "[parameters('vmName')]",`
`135`	`135`	`"linuxConfiguration": {`
	`136`	`+ "disablePasswordAuthentication": false,`
`136`	`137`	`"ssh": {`
`137`	`138`	`"publicKeys": [`
`138`	`139`	`{`
Original file line number	Diff line number	Diff line change
`@@ -116,6 +116,7 @@`
`116`	`116`	`"adminUsername": "[parameters('adminUsername')]",`
`117`	`117`	`"computerName": "[parameters('vmName')]",`
`118`	`118`	`"linuxConfiguration": {`
	`119`	`+ "disablePasswordAuthentication": false,`
`119`	`120`	`"ssh": {`
`120`	`121`	`"publicKeys": [`
`121`	`122`	`{`
Original file line number	Diff line number	Diff line change
`@@ -94,6 +94,7 @@`
`94`	`94`	`"adminUsername": "[parameters('adminUsername')]",`
`95`	`95`	`"computerName": "[parameters('vmName')]",`
`96`	`96`	`"linuxConfiguration": {`
	`97`	`+ "disablePasswordAuthentication": false,`
`97`	`98`	`"ssh": {`
`98`	`99`	`"publicKeys": [`
`99`	`100`	`{`