Thanks Jacques and Pierre for reporting the issue, I;ll dig around this and try to find out actual cause.
Thanks & Regards -- Deepak Dixit www.hotwaxsystems.com On Tue, Nov 24, 2015 at 11:41 AM, Jacques Le Roux < [email protected]> wrote: > I reverted r1715506 at revision: 1716036. The issues I crossed: > > Get to localhost:8080/ecommerce/control/main > Add a product, instead of stating on the main page you get to the cart > page despite having the "Always View Cart After Adding An Item. " not > checked. Then eg: > > Scenario 1 > Use the Recalculate option at top => You get "Your Shopping Cart Empty" > > Scenario 2 > Use the Continue Shopping option at top => your cart is empty > > Scenario 3 > Use the checkout link on top > Login with DemoCustomer > Use the Quick Checkout option > Use the main link on top => your cart is empty > > Jacques > PS: the OFBIZ-VM is down, so no demos are accessible yet, I have asked > infra to restart the VM... > > > Le 23/11/2015 23:17, Jacques Le Roux a écrit : > >> I have also an issue with DemoCustomer, not sure it's the same than >> Pierre's, but it fails >> >> Jacques >> >> Le 23/11/2015 20:52, Pierre Smits a écrit : >> >>> This commit breaks how the ecommerce component works. When I create a new >>> user, and subsequently add a product to the cart, the shop returns to an >>> unknow (anonymous) user and the cart doesn't show the products added. >>> >>> >>> Best regards, >>> >>> >>> >>> Pierre Smits >>> >>> *OFBiz Extensions Marketplace* >>> http://oem.ofbizci.net/oci-2/ >>> >>> On Sat, Nov 21, 2015 at 1:42 PM, <[email protected]> wrote: >>> >>> Author: deepak >>>> Date: Sat Nov 21 12:42:30 2015 >>>> New Revision: 1715506 >>>> >>>> URL: http://svn.apache.org/viewvc?rev=1715506&view=rev >>>> Log: >>>> (OFBIZ-6655) Applied patch from jira issue. >>>> ============================================================ >>>> - Added session cookies config for ecommerce,ecomseo and ecomclone >>>> web.xml. >>>> - Used web-app 3.0 version for web-app >>>> - Re-arrange description and display-name attribute as per the web-app >>>> structure. >>>> >>>> Thanks Rahul for your contribution. >>>> >>>> Modified: >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomclone/WEB-INF/web.xml >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/web.xml >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomseo/WEB-INF/web.xml >>>> >>>> Modified: >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomclone/WEB-INF/web.xml >>>> URL: >>>> >>>> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomclone/WEB-INF/web.xml?rev=1715506&r1=1715505&r2=1715506&view=diff >>>> >>>> >>>> ============================================================================== >>>> --- >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomclone/WEB-INF/web.xml >>>> (original) >>>> +++ >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomclone/WEB-INF/web.xml >>>> Sat Nov 21 12:42:30 2015 >>>> @@ -1,5 +1,4 @@ >>>> <?xml version="1.0"?> >>>> -<!DOCTYPE web-app PUBLIC "-//Sun Microsystems, Inc.//DTD Web >>>> Application >>>> 2.3//EN" "http://java.sun.com/dtd/web-app_2_3.dtd";> >>>> >>>> <!-- >>>> Licensed to the Apache Software Foundation (ASF) under one >>>> @@ -20,41 +19,45 @@ specific language governing permissions >>>> under the License. >>>> --> >>>> >>>> -<web-app> >>>> +<web-app version="3.0" >>>> + xmlns="http://java.sun.com/xml/ns/javaee"; >>>> + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; >>>> + xsi:schemaLocation="http://java.sun.com/xml/ns/javaee >>>> + >>>> http://java.sun.com/xml/ns/javaee/web-app_3_0.xsd";> >>>> <display-name>Apache OFBiz - demostore</display-name> >>>> <description>Cloned Demo Store for the Apache OFBiz >>>> Project</description> >>>> >>>> <context-param> >>>> + <description>A unique ID used to look up the WebSite >>>> entity</description> >>>> <param-name>webSiteId</param-name> >>>> <param-value>WebStoreClone</param-value> >>>> - <description>A unique ID used to look up the WebSite >>>> entity</description> >>>> </context-param> >>>> <context-param> >>>> + <description>A unique name used to identify/recognize the local >>>> dispatcher for the Service Engine</description> >>>> <param-name>localDispatcherName</param-name> >>>> <param-value>ecommerce</param-value> >>>> - <description>A unique name used to identify/recognize the local >>>> dispatcher for the Service Engine</description> >>>> </context-param> >>>> <context-param> >>>> + <description>The Name of the Entity Delegator to use, defined >>>> in >>>> entityengine.xml</description> >>>> <param-name>entityDelegatorName</param-name> >>>> <param-value>default</param-value> >>>> - <description>The Name of the Entity Delegator to use, defined >>>> in >>>> entityengine.xml</description> >>>> </context-param> >>>> <context-param> >>>> + <description>The location of the main-decorator screen to use >>>> for >>>> this webapp; referred to as a context variable in screen def XML >>>> files.</description> >>>> <param-name>mainDecoratorLocation</param-name> >>>> >>>> >>>> <param-value>component://ecommerce/widget/CommonScreens.xml</param-value> >>>> - <description>The location of the main-decorator screen to use >>>> for >>>> this webapp; referred to as a context variable in screen def XML >>>> files.</description> >>>> </context-param> >>>> >>>> <filter> >>>> - <filter-name>CatalogUrlFilter</filter-name> >>>> <display-name>CatalogUrlFilter</display-name> >>>> + <filter-name>CatalogUrlFilter</filter-name> >>>> >>>> <filter-class>org.ofbiz.product.category.CatalogUrlFilter</filter-class> >>>> >>>> >>>> <init-param><param-name>defaultLocaleString</param-name><param-value>en_US</param-value></init-param> >>>> >>>> >>>> <init-param><param-name>redirectUrl</param-name><param-value>/control/main</param-value></init-param> >>>> </filter> >>>> <filter> >>>> - <filter-name>ContextFilter</filter-name> >>>> <display-name>ContextFilter</display-name> >>>> + <filter-name>ContextFilter</filter-name> >>>> >>>> <filter-class>org.ofbiz.webapp.control.ContextFilter</filter-class> >>>> <init-param> >>>> <param-name>disableContextSecurity</param-name> >>>> @@ -92,9 +95,9 @@ under the License. >>>> >>>> >>>> <listener><listener-class>org.ofbiz.webapp.control.LoginEventListener</listener-class></listener> >>>> >>>> <servlet> >>>> - <servlet-name>ControlServlet</servlet-name> >>>> - <display-name>ControlServlet</display-name> >>>> <description>Main Control Servlet</description> >>>> + <display-name>ControlServlet</display-name> >>>> + <servlet-name>ControlServlet</servlet-name> >>>> >>>> <servlet-class>org.ofbiz.webapp.control.ControlServlet</servlet-class> >>>> <load-on-startup>1</load-on-startup> >>>> </servlet> >>>> @@ -108,9 +111,9 @@ under the License. >>>> </servlet> >>>> --> >>>> <servlet> >>>> - <servlet-name>CatalogUrlServlet</servlet-name> >>>> - <display-name>CatalogUrlServlet</display-name> >>>> <description>Catalog (Category/Product) URL >>>> Servlet</description> >>>> + <display-name>CatalogUrlServlet</display-name> >>>> + <servlet-name>CatalogUrlServlet</servlet-name> >>>> >>>> >>>> <servlet-class>org.ofbiz.product.category.CatalogUrlServlet</servlet-class> >>>> <load-on-startup>1</load-on-startup> >>>> </servlet> >>>> @@ -132,6 +135,11 @@ under the License. >>>> >>>> <session-config> >>>> <session-timeout>60</session-timeout> >>>> + <cookie-config> >>>> + <http-only>true</http-only> >>>> + <secure>true</secure> >>>> + </cookie-config> >>>> + <tracking-mode>COOKIE</tracking-mode> >>>> </session-config> >>>> >>>> <welcome-file-list> >>>> >>>> Modified: >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/web.xml >>>> URL: >>>> >>>> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/web.xml?rev=1715506&r1=1715505&r2=1715506&view=diff >>>> >>>> >>>> ============================================================================== >>>> --- >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/web.xml >>>> (original) >>>> +++ >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/WEB-INF/web.xml >>>> Sat Nov 21 12:42:30 2015 >>>> @@ -1,5 +1,4 @@ >>>> <?xml version="1.0"?> >>>> -<!DOCTYPE web-app PUBLIC "-//Sun Microsystems, Inc.//DTD Web >>>> Application >>>> 2.3//EN" "http://java.sun.com/dtd/web-app_2_3.dtd";> >>>> >>>> <!-- >>>> Licensed to the Apache Software Foundation (ASF) under one >>>> @@ -20,48 +19,52 @@ specific language governing permissions >>>> under the License. >>>> --> >>>> >>>> -<web-app> >>>> +<web-app version="3.0" >>>> + xmlns="http://java.sun.com/xml/ns/javaee"; >>>> + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; >>>> + xsi:schemaLocation="http://java.sun.com/xml/ns/javaee >>>> + >>>> http://java.sun.com/xml/ns/javaee/web-app_3_0.xsd";> >>>> <display-name>Apache OFBiz - demostore</display-name> >>>> <description>Demo Store for the Apache OFBiz Project</description> >>>> >>>> <context-param> >>>> + <description>A unique ID used to look up the WebSite >>>> entity</description> >>>> <param-name>webSiteId</param-name> >>>> <param-value>WebStore</param-value> >>>> - <description>A unique ID used to look up the WebSite >>>> entity</description> >>>> </context-param> >>>> <context-param> >>>> + <description>A unique name used to identify/recognize the local >>>> dispatcher for the Service Engine</description> >>>> <param-name>localDispatcherName</param-name> >>>> <param-value>ecommerce</param-value> >>>> - <description>A unique name used to identify/recognize the local >>>> dispatcher for the Service Engine</description> >>>> </context-param> >>>> <context-param> >>>> + <description>The Name of the Entity Delegator to use, defined >>>> in >>>> entityengine.xml</description> >>>> <param-name>entityDelegatorName</param-name> >>>> <param-value>default</param-value> >>>> - <description>The Name of the Entity Delegator to use, defined >>>> in >>>> entityengine.xml</description> >>>> </context-param> >>>> <context-param> >>>> + <description>The location of the main-decorator screen to use >>>> for >>>> this webapp; referred to as a context variable in screen def XML >>>> files.</description> >>>> <param-name>mainDecoratorLocation</param-name> >>>> >>>> >>>> <param-value>component://ecommerce/widget/CommonScreens.xml</param-value> >>>> - <description>The location of the main-decorator screen to use >>>> for >>>> this webapp; referred to as a context variable in screen def XML >>>> files.</description> >>>> </context-param> >>>> <context-param> >>>> - <param-name>forceHttpSession</param-name> >>>> - <param-value>true</param-value> >>>> <description> >>>> Forces the JSESSIONID cookie to be sent via http rather >>>> https, helps prevent lost sessions in web apps that >>>> frequently switch between http and https. >>>> </description> >>>> + <param-name>forceHttpSession</param-name> >>>> + <param-value>true</param-value> >>>> </context-param> >>>> >>>> <filter> >>>> - <filter-name>CatalogUrlFilter</filter-name> >>>> <display-name>CatalogUrlFilter</display-name> >>>> + <filter-name>CatalogUrlFilter</filter-name> >>>> >>>> <filter-class>org.ofbiz.product.category.CatalogUrlFilter</filter-class> >>>> >>>> >>>> <init-param><param-name>defaultLocaleString</param-name><param-value>en_US</param-value></init-param> >>>> </filter> >>>> <filter> >>>> - <filter-name>ContentUrlFilter</filter-name> >>>> <display-name>ContentUrlFilter</display-name> >>>> + <filter-name>ContentUrlFilter</filter-name> >>>> >>>> <filter-class>org.ofbiz.content.content.ContentUrlFilter</filter-class> >>>> <init-param> >>>> <param-name>defaultLocaleString</param-name> >>>> @@ -70,8 +73,8 @@ under the License. >>>> >>>> >>>> <init-param><param-name>viewRequest</param-name><param-value>ViewBlogArticle</param-value></init-param> >>>> </filter> >>>> <filter> >>>> - <filter-name>ContextFilter</filter-name> >>>> <display-name>ContextFilter</display-name> >>>> + <filter-name>ContextFilter</filter-name> >>>> >>>> <filter-class>org.ofbiz.webapp.control.ContextFilter</filter-class> >>>> <init-param> >>>> <param-name>disableContextSecurity</param-name> >>>> @@ -105,8 +108,8 @@ under the License. >>>> >>>> >>>> <listener><listener-class>org.ofbiz.webapp.control.LoginEventListener</listener-class></listener> >>>> >>>> <servlet> >>>> - <servlet-name>ControlServlet</servlet-name> >>>> <display-name>ControlServlet</display-name> >>>> + <servlet-name>ControlServlet</servlet-name> >>>> <description>Main Control Servlet</description> >>>> >>>> <servlet-class>org.ofbiz.webapp.control.ControlServlet</servlet-class> >>>> <load-on-startup>1</load-on-startup> >>>> @@ -121,9 +124,9 @@ under the License. >>>> </servlet> >>>> --> >>>> <servlet> >>>> - <servlet-name>CatalogUrlServlet</servlet-name> >>>> - <display-name>CatalogUrlServlet</display-name> >>>> <description>Catalog (Category/Product) URL >>>> Servlet</description> >>>> + <display-name>CatalogUrlServlet</display-name> >>>> + <servlet-name>CatalogUrlServlet</servlet-name> >>>> >>>> >>>> <servlet-class>org.ofbiz.product.category.CatalogUrlServlet</servlet-class> >>>> <load-on-startup>1</load-on-startup> >>>> </servlet> >>>> @@ -145,6 +148,11 @@ under the License. >>>> >>>> <session-config> >>>> <session-timeout>60</session-timeout> >>>> + <cookie-config> >>>> + <http-only>true</http-only> >>>> + <secure>true</secure> >>>> + </cookie-config> >>>> + <tracking-mode>COOKIE</tracking-mode> >>>> </session-config> >>>> >>>> <welcome-file-list> >>>> >>>> Modified: >>>> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomseo/WEB-INF/web.xml >>>> URL: >>>> >>>> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomseo/WEB-INF/web.xml?rev=1715506&r1=1715505&r2=1715506&view=diff >>>> >>>> >>>> ============================================================================== >>>> --- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomseo/WEB-INF/web.xml >>>> (original) >>>> +++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecomseo/WEB-INF/web.xml >>>> Sat Nov 21 12:42:30 2015 >>>> @@ -1,5 +1,4 @@ >>>> <?xml version="1.0"?> >>>> -<!DOCTYPE web-app PUBLIC "-//Sun Microsystems, Inc.//DTD Web >>>> Application >>>> 2.3//EN" "http://java.sun.com/dtd/web-app_2_3.dtd";> >>>> >>>> <!-- >>>> Licensed to the Apache Software Foundation (ASF) under one >>>> @@ -20,54 +19,58 @@ specific language governing permissions >>>> under the License. >>>> --> >>>> >>>> -<web-app> >>>> +<web-app version="3.0" >>>> + xmlns="http://java.sun.com/xml/ns/javaee"; >>>> + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; >>>> + xsi:schemaLocation="http://java.sun.com/xml/ns/javaee >>>> + >>>> http://java.sun.com/xml/ns/javaee/web-app_3_0.xsd";> >>>> <display-name>Apache OFBiz - demostore</display-name> >>>> <description>Demo Store for the Apache OFBiz Project</description> >>>> >>>> <context-param> >>>> + <description>A unique ID used to look up the WebSite >>>> entity</description> >>>> <param-name>webSiteId</param-name> >>>> <param-value>WebStore</param-value> >>>> - <description>A unique ID used to look up the WebSite >>>> entity</description> >>>> </context-param> >>>> <context-param> >>>> + <description>A unique name used to identify/recognize the local >>>> dispatcher for the Service Engine</description> >>>> <param-name>localDispatcherName</param-name> >>>> <param-value>ecommerce</param-value> >>>> - <description>A unique name used to identify/recognize the local >>>> dispatcher for the Service Engine</description> >>>> </context-param> >>>> <context-param> >>>> + <description>The Name of the Entity Delegator to use, defined >>>> in >>>> entityengine.xml</description> >>>> <param-name>entityDelegatorName</param-name> >>>> <param-value>default</param-value> >>>> - <description>The Name of the Entity Delegator to use, defined >>>> in >>>> entityengine.xml</description> >>>> </context-param> >>>> <context-param> >>>> + <description>The location of the main-decorator screen to use >>>> for >>>> this webapp; referred to as a context variable in screen def XML >>>> files.</description> >>>> <param-name>mainDecoratorLocation</param-name> >>>> >>>> >>>> <param-value>component://ecommerce/widget/CommonScreens.xml</param-value> >>>> - <description>The location of the main-decorator screen to use >>>> for >>>> this webapp; referred to as a context variable in screen def XML >>>> files.</description> >>>> </context-param> >>>> <context-param> >>>> - <param-name>forceHttpSession</param-name> >>>> - <param-value>true</param-value> >>>> <description> >>>> Forces the JSESSIONID cookie to be sent via http rather >>>> https, helps prevent lost sessions in web apps that >>>> frequently switch between http and https. >>>> </description> >>>> + <param-name>forceHttpSession</param-name> >>>> + <param-value>true</param-value> >>>> </context-param> >>>> <context-param> >>>> - <param-name>defaultPage</param-name> >>>> - <param-value>/main</param-value> >>>> <description>Default page uri. Important: please DO add or >>>> remove >>>> /control to match url-pattern of SeoControlServlet. >>>> </description> >>>> + <param-name>defaultPage</param-name> >>>> + <param-value>/main</param-value> >>>> </context-param> >>>> >>>> + <display-name>SeoCatalogUrlFilter</display-name> >>>> <filter> >>>> <filter-name>SeoCatalogUrlFilter</filter-name> >>>> - <display-name>SeoCatalogUrlFilter</display-name> >>>> >>>> >>>> <filter-class>org.ofbiz.product.category.CatalogUrlSeoFilter</filter-class> >>>> >>>> >>>> <init-param><param-name>defaultLocaleString</param-name><param-value>en_US</param-value></init-param> >>>> </filter> >>>> <filter> >>>> - <filter-name>SeoContentUrlFilter</filter-name> >>>> <display-name>SeoContentUrlFilter</display-name> >>>> + <filter-name>SeoContentUrlFilter</filter-name> >>>> >>>> >>>> <filter-class>org.ofbiz.product.category.SeoContentUrlFilter</filter-class> >>>> <init-param> >>>> <param-name>defaultLocaleString</param-name> >>>> @@ -76,8 +79,8 @@ under the License. >>>> >>>> >>>> <init-param><param-name>viewRequest</param-name><param-value>ViewBlogArticle</param-value></init-param> >>>> </filter> >>>> <filter> >>>> - <filter-name>SeoContextFilter</filter-name> >>>> <display-name>SeoContextFilter</display-name> >>>> + <filter-name>SeoContextFilter</filter-name> >>>> >>>> <filter-class>org.ofbiz.product.category.SeoContextFilter</filter-class> >>>> <init-param> >>>> <param-name>disableContextSecurity</param-name> >>>> @@ -111,8 +114,8 @@ under the License. >>>> >>>> >>>> <listener><listener-class>org.ofbiz.webapp.control.LoginEventListener</listener-class></listener> >>>> >>>> <servlet> >>>> - <servlet-name>SeoControlServlet</servlet-name> >>>> <display-name>SeoControlServlet</display-name> >>>> + <servlet-name>SeoControlServlet</servlet-name> >>>> <description>Main SEO Control Servlet</description> >>>> >>>> >>>> <servlet-class>org.ofbiz.product.category.SeoControlServlet</servlet-class> >>>> <load-on-startup>1</load-on-startup> >>>> @@ -127,9 +130,9 @@ under the License. >>>> </servlet> >>>> --> >>>> <servlet> >>>> - <servlet-name>SeoCatalogUrlServlet</servlet-name> >>>> - <display-name>SeoCatalogUrlServlet</display-name> >>>> <description>SEO Catalog (Category/Product) URL >>>> Servlet</description> >>>> + <display-name>SeoCatalogUrlServlet</display-name> >>>> + <servlet-name>SeoCatalogUrlServlet</servlet-name> >>>> >>>> >>>> <servlet-class>org.ofbiz.product.category.SeoCatalogUrlServlet</servlet-class> >>>> <load-on-startup>1</load-on-startup> >>>> </servlet> >>>> @@ -151,6 +154,11 @@ under the License. >>>> >>>> <session-config> >>>> <session-timeout>60</session-timeout> >>>> + <cookie-config> >>>> + <http-only>true</http-only> >>>> + <secure>true</secure> >>>> + </cookie-config> >>>> + <tracking-mode>COOKIE</tracking-mode> >>>> </session-config> >>>> >>>> <welcome-file-list> >>>> >>>> >>>> >>>> >>
