feat(dsn-parser): support + char in driver (#510)

.changeset/popular-coats-beam.md

@@ -0,0 +1,5 @@
+---
+'@httpx/dsn-parser': minor
+---
+
+Support '+' character in driver

examples/nextjs-app/package.json

@@ -31,7 +31,7 @@
     "@types/react": "18.2.17",
     "@types/react-dom": "18.2.7",
     "cross-env": "7.0.3",
-    "eslint": "8.45.0",
+    "eslint": "8.46.0",
     "eslint-config-next": "13.4.12",
     "postcss": "8.4.27",
     "rimraf": "5.0.1",

package.json

@@ -63,7 +63,7 @@
     "cross-env": "7.0.3",
     "docsify": "4.13.1",
     "docsify-cli": "4.4.4",
-    "eslint": "8.45.0",
+    "eslint": "8.46.0",
     "husky": "8.0.3",
     "is-ci": "3.0.1",
     "lint-staged": "13.2.3",

packages/dsn-parser/package.json

@@ -82,7 +82,7 @@
     "browserslist-to-esbuild": "1.2.0",
     "cross-env": "7.0.3",
     "es-check": "7.1.1",
-    "eslint": "8.45.0",
+    "eslint": "8.46.0",
     "get-tsconfig": "4.6.2",
     "npm-run-all": "4.1.5",
     "publint": "0.2.0",

packages/dsn-parser/src/__tests__/parse-dsn.test.ts

@@ -1,6 +1,20 @@
 import { parseDsn } from '../parse-dsn';
 
 describe('parseDsn', () => {
+  describe('when driver is provided with special chars', () => {
+    it.each(['redis', 'Driver55', 'driver-4', '44_driver', 'redis+sentinel'])(
+      `should return %s`,
+      (driver) => {
+        expect(parseDsn(`${driver}://localhost`)).toStrictEqual({
+          success: true,
+          value: {
+            driver,
+            host: 'localhost',
+          },
+        });
+      }
+    );
+  });
   describe('when provided dsn contains all options', () => {
     it('should return the correct parsed params', () => {
       expect(

packages/dsn-parser/src/parse-dsn.ts

@@ -13,7 +13,7 @@ import {
 import { parseQueryParams } from './query-param-parser';
 
 const dsnRegexp =
-  /^(?<driver>([\w-]+)):\/\/((?<user>[^/:]{1,200})?(:(?<pass>.{0,250}))?@)?(?<host>[^/:]{1,400}?)(:(?<port>\d+)?)?(\/(?<db>([.#@$\w-])+))?(\?(?<params>.+))?$/;
+  /^(?<driver>([\w+-]{1,45})):\/\/((?<user>[^/:]{1,200})?(:(?<pass>.{0,250}))?@)?(?<host>[^/:]{1,400}?)(:(?<port>\d+)?)?(\/(?<db>([.#@$\w-])+))?(\?(?<params>.{1,8196}))?$/;
 
 const defaultOptions = {
   lowercaseDriver: false,

packages/exception/package.json

@@ -107,7 +107,7 @@
     "@size-limit/file": "8.2.6",
     "@size-limit/webpack": "8.2.6",
     "@size-limit/webpack-why": "8.2.6",
-    "@swc/core": "1.3.71",
+    "@swc/core": "1.3.72",
     "@types/jest": "29.5.3",
     "@types/node": "20.4.5",
     "@types/statuses": "2.0.1",
@@ -117,13 +117,13 @@
     "cross-env": "7.0.3",
     "error-cause-polyfill": "2.0.0",
     "es-check": "7.1.1",
-    "eslint": "8.45.0",
+    "eslint": "8.46.0",
     "get-tsconfig": "4.6.2",
     "jest": "29.6.2",
     "npm-run-all": "4.1.5",
     "publint": "0.1.16",
     "rimraf": "5.0.1",
-    "rollup": "3.26.3",
+    "rollup": "3.27.0",
     "rollup-plugin-dts": "5.3.0",
     "rollup-plugin-swc3": "0.9.0",
     "size-limit": "8.2.6",

packages/json-api/package.json

@@ -84,7 +84,7 @@
     "browserslist-to-esbuild": "1.2.0",
     "cross-env": "7.0.3",
     "es-check": "7.1.1",
-    "eslint": "8.45.0",
+    "eslint": "8.46.0",
     "get-tsconfig": "4.6.2",
     "npm-run-all": "4.1.5",
     "publint": "0.2.0",