[ofbiz-framework] branch trunk updated: Fixed: Send Confirmation Mail Request Problem (OFBIZ-12674)

[jleroux]

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new d48712546d Fixed: Send Confirmation Mail Request Problem (OFBIZ-12674)
d48712546d is described below

commit d48712546d63ee0e0c0d9bd9d675f62e6d0b3f76
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Mon Sep 12 09:16:42 2022 +0200

    Fixed: Send Confirmation Mail Request Problem (OFBIZ-12674)
    
    The sendconfirmationmail request exists but RequestHandler receives
    "/sendconfirmationmail/getJSONuiLabelArray"
    
    Solution: this is a known nested request, we can bypass
    throwRequestHandlerExceptionOnMissingLocalRequest
    
    Thanks: Ingo Wolfmayr for report
---
 .../main/java/org/apache/ofbiz/webapp/control/RequestHandler.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/RequestHandler.java
 
b/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/RequestHandler.java
index 18ec08d8d5..8c9f715183 100644
--- 
a/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/RequestHandler.java
+++ 
b/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/RequestHandler.java
@@ -395,9 +395,9 @@ public final class RequestHandler {
         Collection<RequestMap> rmaps = resolveURI(ccfg, request);
         if (rmaps.isEmpty()) {
             if (throwRequestHandlerExceptionOnMissingLocalRequest) {
-                if (path.contains("/checkLogin/")) {
-                    // Nested requests related with checkLogin uselessly 
clutter the log. There is nothing to worry about, better remove this wrong
-                    // error message.
+                if (path.contains("/checkLogin/") || 
path.contains("/sendconfirmationmail/")) {
+                    // Nested requests related with checkLogin and 
sendconfirmationmail are OK.
+                    // There is nothing to worry about, better remove these 
wrong errors messages.
                     return;
                 } else if (path.contains("/images/") || 
path.contains("d.png")) {
                     if (Debug.warningOn()) {