fix(common): addLeadingSlash optional group support

nestjs · Jan 31, 2025 · f81ce5f · f81ce5f
1 parent 13290e9
commit f81ce5f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/common/test/utils/shared.utils.spec.ts b/packages/common/test/utils/shared.utils.spec.ts
@@ -117,9 +117,11 @@ describe('Shared utils', () => {
   describe('addLeadingSlash', () => {
     it('should return the validated path ("add / if not exists")', () => {
       expect(addLeadingSlash('nope')).to.be.eql('/nope');
+      expect(addLeadingSlash('{:nope}')).to.be.eql('/{:nope}');
     });
     it('should return the same path', () => {
       expect(addLeadingSlash('/nope')).to.be.eql('/nope');
+      expect(addLeadingSlash('{/:nope}')).to.be.eql('{/:nope}');
     });
     it('should return empty path', () => {
       expect(addLeadingSlash('')).to.be.eql('');

diff --git a/packages/common/utils/shared.utils.ts b/packages/common/utils/shared.utils.ts
@@ -25,7 +25,7 @@ export const isPlainObject = (fn: any): fn is object => {
 
 export const addLeadingSlash = (path?: string): string =>
   path && typeof path === 'string'
-    ? path.charAt(0) !== '/'
+    ? path.charAt(0) !== '/' && path.substring(0, 2) !== '{/'
       ? '/' + path
       : path
     : '';