yykaue opened a new pull request, #6969: URL: https://github.com/apache/shenyu/pull/6969
Fixes #6805. ## Summary - Precompute the default module names for the Divide and WebSocket context decorators. - Remove per-request `String.format` calls from the request path. - Add an assertion covering the default Divide module name. - Preserve the existing module naming behavior. `Optional.orElse(...)` evaluates its fallback value eagerly, so the previous `String.format` was executed on every request, even when metadata contained an application name. Using static constants avoids this unnecessary formatting and allocation. @Aias00, could you please help review this PR when you have time? Thank you! Make sure that: - [ ] You have read the [contribution guidelines](https://shenyu.apache.org/community/contributor-guide). - [x] You submit test cases (unit or integration tests) that back your changes. - [ ] Your local test passed `./mvnw clean install -Dmaven.javadoc.skip=true`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
