You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Pierre Smits <pi...@gmail.com> on 2015/11/23 20:52:51 UTC

Re: svn commit: r1715506 - in /ofbiz/trunk/specialpurpose/ecommerce/webapp: ecomclone/WEB-INF/web.xml ecommerce/WEB-INF/web.xml ecomseo/WEB-INF/web.xml

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, <de...@apache.org> 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>
>
>
>

Re: svn commit: r1715506 - in /ofbiz/trunk/specialpurpose/ecommerce/webapp: ecomclone/WEB-INF/web.xml ecommerce/WEB-INF/web.xml ecomseo/WEB-INF/web.xml

Posted by Deepak Dixit <de...@hotwaxsystems.com>.
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 <
jacques.le.roux@les7arts.com> 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, <de...@apache.org> 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>
>>>>
>>>>
>>>>
>>>>
>>

Re: svn commit: r1715506 - in /ofbiz/trunk/specialpurpose/ecommerce/webapp: ecomclone/WEB-INF/web.xml ecommerce/WEB-INF/web.xml ecomseo/WEB-INF/web.xml

Posted by Jacques Le Roux <ja...@les7arts.com>.
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, <de...@apache.org> 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>
>>>
>>>
>>>
>

Re: svn commit: r1715506 - in /ofbiz/trunk/specialpurpose/ecommerce/webapp: ecomclone/WEB-INF/web.xml ecommerce/WEB-INF/web.xml ecomseo/WEB-INF/web.xml

Posted by Jacques Le Roux <ja...@les7arts.com>.
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, <de...@apache.org> 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>
>>
>>
>>