juicewcode commented on code in PR #6951:
URL: https://github.com/apache/shenyu/pull/6951#discussion_r4015021738
##########
shenyu-plugin/shenyu-plugin-rewrite/src/main/java/org/apache/shenyu/plugin/rewrite/RewritePlugin.java:
##########
@@ -76,7 +76,7 @@ protected Mono<Void> doExecute(final ServerWebExchange
exchange, final ShenyuPlu
&& ThreadLocalRandom.current().nextInt(100) < percentage) {
rewriteUri = rewriteHandle.getReplace().contains("{") &&
rewriteHandle.getRegex().contains("{")
? PathMatchUtils.replaceAll(rewriteHandle.getReplace(),
rewriteHandle.getRegex().substring(rewriteHandle.getRegex().indexOf("{")),
-
rewriteUri.substring(rewriteHandle.getRegex().indexOf("{")))
+
rewriteUri.substring(rewriteHandle.getRegex().indexOf("{")))
Review Comment:
Hi, the tab change here is just from resolving a merge conflict between my
branch and master. My actual code changes are in the first commit
([f4a3193](https://github.com/apache/shenyu/pull/6951/commits/f4a3193637d16251d7d6c03ab9bf7bbdf04c055a))
of this PR.It is likely that the same changes were already merged into master,
so they no longer appear in the diff.
--
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]
