feat: reduce dsn regexp character lengths

belgattitude · Oct 20, 2023 · 8681dc7 · 8681dc7
1 parent 34432cc
commit 8681dc7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/slimy-pots-glow.md b/.changeset/slimy-pots-glow.md
@@ -0,0 +1,5 @@
+---
+'@httpx/dsn-parser': minor
+---
+
+Reduce dsn regexp character lengths (reduce complexity)
diff --git a/packages/dsn-parser/src/parse-dsn.ts b/packages/dsn-parser/src/parse-dsn.ts
@@ -13,8 +13,8 @@ import {
 import { parseQueryParams } from './query-param-parser';
 
 const dsnRegexp =
-  // eslint-disable-next-line regexp/no-unused-capturing-group
-  /^(?<driver>([\w+-]{1,45})):\/\/((?<user>[^/:]{1,200})?(:(?<pass>.{0,250}))?@)?(?<host>[^/:]{1,400}?)(:(?<port>\d+)?)?(\/(?<db>([.#@$\w-])+))?(\?(?<params>.{1,8196}))?$/;
+  // eslint-disable-next-line regexp/no-unused-capturing-group,regexp/no-misleading-capturing-group
+  /^(?<driver>([\w+-]{1,40})):\/\/((?<user>[^/:]{1,200})?(:(?<pass>.{0,200}))?@)?(?<host>[^/:]{1,200}?)(:(?<port>\d+)?)?(\/(?<db>([.#@$\w-])+))?(\?(?<params>.{1,1000}))?$/;
 
 const defaultOptions = {
   lowercaseDriver: false,