ofbiz-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacques Le Roux (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OFBIZ-10304) The "request" attribute type of the userLogin service is wrong
Date Mon, 02 Apr 2018 08:30:00 GMT

    [ https://issues.apache.org/jira/browse/OFBIZ-10304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16422018#comment-16422018

Jacques Le Roux commented on OFBIZ-10304:

This issue should not be closed before OFBIZ-10036 is resolved. For now we have only workaounds.
Only upgrading to Tomcat 9 (or downgrading to Tomcat 8, which I guess is not wanted) will
correctly fix this issue.

> The "request" attribute type of the userLogin service is wrong
> --------------------------------------------------------------
>                 Key: OFBIZ-10304
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10304
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: Trunk, 17.12.01
>            Reporter: Jacques Le Roux
>            Assignee: Jacques Le Roux
>            Priority: Blocker
>             Fix For: 17.12.01
>         Attachments: OFBIZ-10304.patch
> I commited [http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/webapp/src/main/java/org/apache/ofbiz/webapp/control/ContextFilter.java?r1=1813679&r2=1813678&pathrev=1813679]
which I guess forced (or allowed?) James Yong to use the javax.servlet.http.HttpServletRequest
as type of "request" attribute of the userLogin service at [http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/servicedef/services.xml?r1=1819133&r2=1819132&pathrev=1819133]
> Else it would (should?) have been
>  <attribute name="request" mode="IN" type="org.apache.catalina.connector.RequestFacade"
> Now I need to revert/remove the wrapper in ContextFilter which is useless and silly (my
> {color:#ff0000}-After a deeper investigation today (saturday 2019-03-24) the right solution
is now attached as a patch-
>  UPDATE: we had a discussion on this subject at [https://markmail.org/message/4jwrnbhpapb4ncrd]
and Scott suggested to rather temporary use the <type-validate> child element of the
<service> <attribute> element. Temporary because the definitive solution (w/o
using <type-validate>) could be either to update to Tomcat 9 (preferred) or even downgrade
to Tomcat 8. I guess nobody really want the later because only the trunk and not yet released
R17.12 branch are concerned.{color}

This message was sent by Atlassian JIRA

View raw message