Add multi-byte character check

sqids · Sep 9, 2023 · ac8145d · ac8145d
1 parent cd5ca9b
commit ac8145d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/Sqids/SqidsEncoder.cs b/src/Sqids/SqidsEncoder.cs
@@ -81,6 +81,12 @@ public SqidsEncoder(SqidsOptions options)
 				$"The alphabet must contain at least {MinAlphabetLength} characters."
 			);
 
+		if (Encoding.UTF8.GetByteCount(options.Alphabet) != options.Alphabet.Length)
+			throw new ArgumentOutOfRangeException(
+				nameof(options.MinLength),
+				$"The alphabet must not contain multi-byte characters."
+			);
+
 		if (options.MinLength < 0 || options.MinLength > MaxMinLength)
 			throw new ArgumentOutOfRangeException(
 				nameof(options.MinLength),

diff --git a/test/Sqids.Tests/AlphabetTests.cs b/test/Sqids.Tests/AlphabetTests.cs
@@ -39,6 +39,7 @@ int[] numbers
 
 	[TestCase("aabcdefg")] // NOTE: Repeated characters
 	[TestCase("ab")] // NOTE: Too short
+	[TestCase("ë1092")] // NOTE: Contains a multi-byte character
 	public void Instantiate_WithInvalidAlphabet_Throws(string invalidAlphabet)
 	{
 #if NET7_0_OR_GREATER