(ofbiz-framework) branch release18.12 updated: Revert "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 9869bed35a Revert "Fixed: Prevents to uselessly clutter the logs up 
with SetTimeZoneFromBrowser errors (OFBIZ-13061)"
9869bed35a is described below

commit 9869bed35a9851d5574fd7df395f56c36713e84b
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Wed May 1 15:37:48 2024 +0200

    Revert "Fixed: Prevents to uselessly clutter the logs up with 
SetTimeZoneFromBrowser errors (OFBIZ-13061)"
    
    This reverts commit 135d137a2483f94fe8e95fcdb39a18f1c0b95963.
    This change should not have been backported
---
 .../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 71f0ec93ef..5526602674 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 && 
!modelService.name.equals("SetTimeZoneFromBrowser")) {
+                    if (modelService.auth && userLogin == null) {
                         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 && 
!service.name.equals("SetTimeZoneFromBrowser")) {
+                if (service.auth && userLogin == null) {
                     throw new ServiceAuthException("User authorization is 
required for this service: " + service.name + service.debugInfo());
                 }