OpenSSHUtils.psm1 Uses Non-standard "Quiet" Switch Name (and non-standard Functionality) for PowerShell standard "Confirm" Switch

[DarwinJS]

The -Quiet switch in OpenSSHUtils.psm1 appear to be functioning in ways that mimic the standard "Confirm" switch of PowerShell (example:

Win32-OpenSSH/contrib/win32/openssh/OpenSSHUtils.psm1

Lines 184 to 195 in 2c3a1a9

	if (-not $Quiet) {
	$warning = "Current owner: '$($acl.Owner)'. '$($Owners[0])' should own $FilePath."
	Do {
	Write-Warning $warning
	$input = Read-Host -Prompt "Shall I set the file owner? [Yes] Y; [No] N (default is `"Y`")"
	if([string]::IsNullOrEmpty($input))
	{
	$input = 'Y'
	}
	} until ($input -match "^(y(es)?|N(o)?)$")
	$result = $Matches[0]
	}

)

The standard way is to use "SupportsShouldProcess" in the CMDLET binding

However, I personally do not like the degraded readability and what seems like some counter-intuitive logic in my code when I put the actual code changes to support "SupportsShouldProcess".

In this case I would probably cheat and just add a parameter "Confirm". Also this approach would be able to retain your timeout - which I like and I believe is NOT part of standard ShouldProcess support.

[bingbing8]

@DarwinJS , I added commit to the fix of this issue. Since we are discussing how to handle the future breaking changes and release of scripts, I can't submit the changes soon. Below include multiple fixes if you are interested:

1. use valid verb,
2. psd1 file to specify version,
3. -confirm support,
4. workaround the error "the security identifier is not allowed to be the owner of this object" on downlevel machine.