PascalParams, camelVariables, enhance consistency in readme & scripts

Author: TimHess

FileShares/README.md

@@ -75,9 +75,12 @@ Before deploying the app, you must create an entry in CredHub to contain the cre
 > The [cf-create-service.ps1](scripts/cf-create-service.ps1) script requires PowerShell 7 or later.
 
 1. Run [cf-create-service.ps1](scripts/cf-create-service.ps1) to create a service instance in CredHub, using parameters to set the required values:
-   * `-NetworkAddress \\<hostname>\<sharename>` - UNC path of the fileshare
-   * `-UserName <username>` - the username for accessing the fileshare (can include domain, e.g., `DOMAIN\username`)
-   * `-Password <password>` - the password for accessing the fileshare
+   * `-NetworkAddress \\<hostname>\<sharename>` - UNC path to the network share (required). For example: `\\localhost\steeltoe_network_share`
+   * `-UserName <username>` - the username for accessing the fileshare, can include domain (e.g., `DOMAIN\username`) (required)
+   * `-Password <password>` - the password for accessing the fileshare (required)
+   * `-ServiceName credhub` - the name of the service
+   * `-ServicePlan default` - the service plan
+   * `-ServiceInstanceName sampleNetworkShare` - the name of the service instance
 
 ### Deploy the app
 

FileShares/scripts/add-user-and-share.ps1

@@ -2,12 +2,13 @@
 #Requires -Modules Microsoft.PowerShell.LocalAccounts, SmbShare
 
 Param(
-	[string]$ShareName = "steeltoe_network_share",
-	[string]$SharePath = "c:\steeltoe_network_share",
-	[string]$UserName = "shareWriteUser",
-	[string]$Password = "thisIs1Pass!"
+    [Parameter(Mandatory = $false)][string]$ShareName = "steeltoe_network_share",
+    [Parameter(Mandatory = $false)][string]$SharePath = "c:\steeltoe_network_share",
+    [Parameter(Mandatory = $false)][string]$UserName = "shareWriteUser",
+    [Parameter(Mandatory = $false)][string]$Password = "thisIs1Pass!"
 )
 $ErrorActionPreference = "Stop"
+
 if ($PSVersionTable.PSVersion.Major -lt 6)
 {
     Write-Output "Running in Windows PowerShell (version < 6)"
@@ -18,7 +19,7 @@ else
     Add-Type -AssemblyName System.Management.Automation
     Import-Module Microsoft.PowerShell.LocalAccounts -SkipEditionCheck
 }
-$SecurePassword = ConvertTo-SecureString -String $Password -AsPlainText -Force
+$securePassword = ConvertTo-SecureString -String $Password -AsPlainText -Force
 
 if (Get-LocalUser -Name $UserName -ErrorAction SilentlyContinue)
 {
@@ -28,7 +29,7 @@ else
 {
     Write-Host "Creating local user $UserName..."
     New-LocalUser $UserName `
-        -Password $SecurePassword `
+        -Password $securePassword `
         -FullName "SMB ReadWrite" `
         -Description "For write access to $ShareName" | Out-Null
     Write-Host "Done creating user."

FileShares/scripts/cf-create-service.ps1

@@ -2,11 +2,11 @@
 
 Param(
     [Parameter(Mandatory = $true, HelpMessage = "UNC path to the network share. For example: '\\localhost\steeltoe_network_share'")][string]$NetworkAddress,
-    [Parameter(Mandatory=$true)][string]$UserName,
-    [Parameter(Mandatory=$true)][string]$Password,
-    [string]$ServiceName = "credhub",
-    [string]$ServicePlan = "default",
-    [string]$ServiceInstanceName = "sampleNetworkShare"
+    [Parameter(Mandatory = $true)][string]$UserName,
+    [Parameter(Mandatory = $true)][string]$Password,
+    [Parameter(Mandatory = $false)][string]$ServiceName = "credhub",
+    [Parameter(Mandatory = $false)][string]$ServicePlan = "default",
+    [Parameter(Mandatory = $false)][string]$ServiceInstanceName = "sampleNetworkShare"
 )
 $ErrorActionPreference = "Stop"
 
@@ -17,12 +17,12 @@ $params = @{
     username = $UserName
     password = $Password
 }
-$ParamJSON = $params | ConvertTo-Json -Compress
+$jsonParams = $params | ConvertTo-Json -Compress
 
 # Create a redacted copy of the parameters for logging so the password is not exposed
 $redactedParams = $params.Clone()
 $redactedParams['password'] = 'REDACTED'
-$ParamJSONRedacted = $redactedParams | ConvertTo-Json -Compress
+$redactedJsonParams = $redactedParams | ConvertTo-Json -Compress
 
-Write-Host "cf create-service $ServiceName $ServicePlan $ServiceInstanceName -c $ParamJSONRedacted -t $ServiceInstanceName"
-cf create-service $ServiceName $ServicePlan $ServiceInstanceName -c $ParamJSON -t $ServiceInstanceName
+Write-Host "cf create-service $ServiceName $ServicePlan $ServiceInstanceName -c $redactedJsonParams -t $ServiceInstanceName"
+cf create-service $ServiceName $ServicePlan $ServiceInstanceName -c $jsonParams -t $ServiceInstanceName

FileShares/scripts/remove-user-and-share.ps1

@@ -2,11 +2,12 @@
 #Requires -Modules Microsoft.PowerShell.LocalAccounts, SmbShare
 
 Param(
-    [string]$ShareName = "steeltoe_network_share",
-    [string]$SharePath = "c:\steeltoe_network_share",
-    [string]$UserName = "shareWriteUser"
+    [Parameter(Mandatory = $false)][string]$ShareName = "steeltoe_network_share",
+    [Parameter(Mandatory = $false)][string]$SharePath = "c:\steeltoe_network_share",
+    [Parameter(Mandatory = $false)][string]$UserName = "shareWriteUser"
 )
 $ErrorActionPreference = "Stop"
+
 if ($PSVersionTable.PSVersion.Major -lt 6)
 {
     Write-Output "Running in Windows PowerShell (version < 6)"
@@ -17,6 +18,7 @@ else
     Add-Type -AssemblyName System.Management.Automation
     Import-Module Microsoft.PowerShell.LocalAccounts -SkipEditionCheck
 }
+
 if (Get-SmbShare $ShareName -ErrorAction SilentlyContinue)
 {
     Remove-SmbShare -Name $ShareName