fix a bug in plugin Redirector. (#2198)

tanglong3bf · web-flow · commit 882c1d9ecd35 · 2024-11-16T15:04:57.000+08:00
diff --git a/lib/src/Redirector.cc b/lib/src/Redirector.cc
@@ -30,17 +30,17 @@ void Redirector::initAndStart(const Json::Value &config)
             }
             std::string protocol, host;
             bool pathChanged{false};
+            for (auto &handler : thisPtr->pathRewriteHandlers_)
+            {
+                pathChanged |= handler(req);
+            }
             for (auto &handler : thisPtr->handlers_)
             {
                 if (!handler(req, protocol, host, pathChanged))
                 {
                     return HttpResponse::newNotFoundResponse(req);
                 }
             }
-            for (auto &handler : thisPtr->pathRewriteHandlers_)
-            {
-                pathChanged |= handler(req);
-            }
             if (!protocol.empty() || !host.empty() || pathChanged)
             {
                 std::string url;

Original file line number	Diff line number	Diff line change
`@@ -30,17 +30,17 @@ void Redirector::initAndStart(const Json::Value &config)`
`30`	`30`	`}`
`31`	`31`	`std::string protocol, host;`
`32`	`32`	`bool pathChanged{false};`
	`33`	`+ for (auto &handler : thisPtr->pathRewriteHandlers_)`
	`34`	`+ {`
	`35`	`+ pathChanged \|= handler(req);`
	`36`	`+ }`
`33`	`37`	`for (auto &handler : thisPtr->handlers_)`
`34`	`38`	`{`
`35`	`39`	`if (!handler(req, protocol, host, pathChanged))`
`36`	`40`	`{`
`37`	`41`	`return HttpResponse::newNotFoundResponse(req);`
`38`	`42`	`}`
`39`	`43`	`}`
`40`		`- for (auto &handler : thisPtr->pathRewriteHandlers_)`
`41`		`- {`
`42`		`- pathChanged \|= handler(req);`
`43`		`- }`
`44`	`44`	`if (!protocol.empty() \|\| !host.empty() \|\| pathChanged)`
`45`	`45`	`{`
`46`	`46`	`std::string url;`