(ofbiz-framework) branch release18.12 updated: Fixed: Prevents to uselessly clutter the logs up with SetTimeZoneFromBrowser errors (OFBIZ-13061)

[jleroux]

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

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


The following commit(s) were added to refs/heads/release18.12 by this push:
     new 135d137a24 Fixed: Prevents to uselessly clutter the logs up with 
SetTimeZoneFromBrowser errors (OFBIZ-13061)
135d137a24 is described below

commit 135d137a2483f94fe8e95fcdb39a18f1c0b95963
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Wed May 1 15:13:49 2024 +0200

    Fixed: Prevents to uselessly clutter the logs up with 
SetTimeZoneFromBrowser errors (OFBIZ-13061)
    
    The previous commits were not enough as I feared. This time we should not 
see
    useless "error" information while running services.
    
    Conflicts hanled by hands
---
 .../src/main/java/org/apache/ofbiz/service/ServiceDispatcher.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/framework/service/src/main/java/org/apache/ofbiz/service/ServiceDispatcher.java
 
b/framework/service/src/main/java/org/apache/ofbiz/service/ServiceDispatcher.java
index 5526602674..71f0ec93ef 100644
--- 
a/framework/service/src/main/java/org/apache/ofbiz/service/ServiceDispatcher.java
+++ 
b/framework/service/src/main/java/org/apache/ofbiz/service/ServiceDispatcher.java
@@ -370,7 +370,7 @@ public class ServiceDispatcher {
                     context = checkAuth(localName, context, modelService);
                     GenericValue userLogin = (GenericValue) 
context.get("userLogin");
 
-                    if (modelService.auth && userLogin == null) {
+                    if (modelService.auth && userLogin == null && 
!modelService.name.equals("SetTimeZoneFromBrowser")) {
                         rs.setEndStamp();
                         throw new ServiceAuthException("User authorization is 
required for this service: " + modelService.name + modelService.debugInfo());
                     }
@@ -717,7 +717,7 @@ public class ServiceDispatcher {
                 context = checkAuth(localName, context, service);
                 Object userLogin = context.get("userLogin");
 
-                if (service.auth && userLogin == null) {
+                if (service.auth && userLogin == null && 
!service.name.equals("SetTimeZoneFromBrowser")) {
                     throw new ServiceAuthException("User authorization is 
required for this service: " + service.name + service.debugInfo());
                 }