[ofbiz-framework] branch release22.01 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 release22.01
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


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

commit 64eb9607e06050ec9641ea88dc7f30f42b287546
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 877a5de387..fadf916001 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
@@ -394,9 +394,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()) {