fix: feedbacks

Author: coratgerl

9.0.0.md

@@ -48,7 +48,9 @@ rateLimit: {
 - Review your custom cloud routes and ensure they use the new path-to-regexp v8 syntax.
 - Update any rate-limiting configurations to use the new route path format.
 - Test your application to ensure all routes work as expected with the new syntax.
-- **Consult the [Express 5 Migration Guide](https://expressjs.com/en/guide/migrating-5.html#path-syntax) for more details on the new path syntax.**
+
+> [!Note]
+> Consult the [path-to-regexp v8 docs](https://github.com/pillarjs/path-to-regexp) and the [Express 5 migration guide](https://expressjs.com/en/guide/migrating-5.html#path-syntax) for more details on the new path syntax.
 
 ### Related Pull Request
-- [#9942: feat: Bump path-to-regexp to v8](https://github.com/parse-community/parse-server/pull/9942)
+- [#9942](https://github.com/parse-community/parse-server/pull/9942)

spec/RateLimit.spec.js

@@ -481,14 +481,6 @@ describe('rate limit', () => {
     expect(() =>
       validateRateLimit({ rateLimit: [{ requestTimeWindow: 3, requestCount: 'abc' }] })
     ).toThrow('rateLimit.requestPath must be defined');
-    expect(() =>
-      validateRateLimit({ rateLimit: [{ requestPath: '/*' }] })
-    ).toThrow(`rateLimit.requestPath "/*" uses deprecated wildcard syntax. ` +
-      `Please update to path-to-regexp v8 syntax. Examples:\n` +
-      `  Old: "/functions/*" → New: "/functions/*path"\n` +
-      `  Old: "/classes/*" → New: "/classes/*path"\n` +
-      `  Old: "*" → New: "*path"\n` +
-      `See parameter name on the express migration guide.`);
     await expectAsync(
       reconfigureServer({
         rateLimit: [{ requestTimeWindow: 3, requestCount: 1, path: 'abc', requestPath: 'a' }],

src/Config.js

@@ -689,19 +689,6 @@ export class Config {
         throw `rateLimit.requestPath must be a string`;
       }
 
-      // Validate path-to-regexp v8 syntax
-      // Check for common old syntax patterns
-      const oldWildcardPattern = /(?:^|\/)\*(?:\/|$)/;  // Matches /* or * at start/end
-      const nakedWildcard = /^[\s]*\*[\s]*$/;  // Matches bare *
-      if (oldWildcardPattern.test(option.requestPath) || nakedWildcard.test(option.requestPath)) {
-        throw `rateLimit.requestPath "${option.requestPath}" uses deprecated wildcard syntax. ` +
-              `Please update to path-to-regexp v8 syntax. Examples:\n` +
-              `  Old: "/functions/*" → New: "/functions/*path"\n` +
-              `  Old: "/classes/*" → New: "/classes/*path"\n` +
-              `  Old: "*" → New: "*path"\n` +
-              `See parameter name on the express migration guide.`;
-      }
-
       // Validate that the path is valid path-to-regexp syntax
       try {
         pathToRegexp(option.requestPath);

src/Options/index.js

@@ -350,7 +350,7 @@ export interface ParseServerOptions {
 }
 
 export interface RateLimitOptions {
-  /* The path of the API route to be rate limited. Route paths, in combination with a request method, define the endpoints at which requests can be made. Route paths can be strings or string patterns following path-to-regexp v8 syntax. Wildcards must be named (e.g., `/*path` instead of `/*`). Examples: `/functions/*path`, `/classes/MyClass/*path`, `/*path`. See: https://github.com/pillarjs/path-to-regexp */
+  /* The path of the API route to be rate limited. Route paths, in combination with a request method, define the endpoints at which requests can be made. Route paths can be strings or string patterns following <a href="https://github.com/pillarjs/path-to-regexp">path-to-regexp v8</a> syntax. */
   requestPath: string;
   /* The window of time in milliseconds within which the number of requests set in `requestCount` can be made before the rate limit is applied. */
   requestTimeWindow: ?number;