fix(util-endpoints): reject non-ASCII input in substring per Smithy spec

Author: TrevorBurnham

packages/util-endpoints/src/lib/substring.spec.ts

@@ -12,6 +12,12 @@ describe(substring.name, () => {
     it("when input.length < stop", () => {
       expect(substring("", 0, 1, false)).toBeNull();
     });
+
+    it("when input contains non-ASCII characters", () => {
+      expect(substring("abc\u0080", 0, 3, false)).toBeNull();
+      expect(substring("abcé", 0, 3, false)).toBeNull();
+      expect(substring("ab日c", 0, 3, false)).toBeNull();
+    });
   });
 
   it("returns substring", () => {

packages/util-endpoints/src/lib/substring.ts

@@ -5,7 +5,7 @@
  * The length of the returned string will always be stop-start.
  */
 export const substring = (input: string, start: number, stop: number, reverse: boolean): string | null => {
-  if (start >= stop || input.length < stop) {
+  if (start >= stop || input.length < stop || /[^\u0000-\u007f]/.test(input)) {
     return null;
   }
   if (!reverse) {