From 914a140e736d397217cd6ecb3835166a6cd97119 Mon Sep 17 00:00:00 2001 From: Ardis Lu Date: Thu, 23 Dec 2021 23:41:04 -0800 Subject: [PATCH] Make input strings mandatory It doesn't make sense to default to null or empty string, it should explicitly ask for input. --- ArdisUtilities/ArdisUtilities.psm1 | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ArdisUtilities/ArdisUtilities.psm1 b/ArdisUtilities/ArdisUtilities.psm1 index 40393ef..34f3d9c 100644 --- a/ArdisUtilities/ArdisUtilities.psm1 +++ b/ArdisUtilities/ArdisUtilities.psm1 @@ -120,7 +120,7 @@ function ConvertTo-Base64String { [OutputType([String])] [CmdletBinding()] param( - [Parameter(ValueFromPipeline)] + [Parameter(Mandatory, ValueFromPipeline)] [String]$InputString ) @@ -168,8 +168,8 @@ function Get-StringHash { [OutputType('Microsoft.PowerShell.Commands.FileHashInfo')] [CmdletBinding()] param( - [Parameter(ValueFromPipeline)] - [String]$InputString = '', + [Parameter(Mandatory, ValueFromPipeline)] + [String]$InputString, [ValidateSet('SHA1', 'SHA256', 'SHA384', 'SHA512', 'MD5')] [String]$Algorithm = 'SHA1' -- 2.47.3