Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-5919

CatalogUrlServlet should return a 404 HTTP status if the category or product does not exist

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Release Branch 12.04, Release Branch 13.07, Trunk
    • Fix Version/s: 14.12.01, 16.11.01
    • Component/s: ecommerce
    • Labels:
      None
    • Sprint:
      Bug Crush Event - 21/2/2015

      Description

      To test: http://demo-trunk-ofbiz.apache.org/ecommerce/products/dropShip/dropShip1-err

      Doesn't work: Says catalog not found, but the acces_log shows http status 200.

        Activity

        Hide
        lektran Scott Gray added a comment -

        What exactly is the issue Jacques? dropShip1-err doesn't exist...

        Show
        lektran Scott Gray added a comment - What exactly is the issue Jacques? dropShip1-err doesn't exist...
        Show
        jacques.le.roux Jacques Le Roux added a comment - It comes from here https://issues.apache.org/jira/browse/OFBIZ-4390?focusedCommentId=13162213
        Hide
        lektran Scott Gray added a comment -

        Okay thanks I've changed the title to something a bit more useful.

        Show
        lektran Scott Gray added a comment - Okay thanks I've changed the title to something a bit more useful.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Scott!

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Scott!
        Hide
        vishal.chhabria Vishal Chhabria added a comment - - edited

        IMO, 404 should be returned in case any url or request does not exist but /ecommerce/products/product-id is used to find the product with given productId.
        So I think this is the correct behaviour and CatalogUrlServlet should not return 404 HTTP status if any product is not found.

        Show
        vishal.chhabria Vishal Chhabria added a comment - - edited IMO, 404 should be returned in case any url or request does not exist but /ecommerce/products/product-id is used to find the product with given productId. So I think this is the correct behaviour and CatalogUrlServlet should not return 404 HTTP status if any product is not found.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Though unrelated (this bug report is anterior to the typo fixed by OFBIZ-6650) this has been fixed by OFBIZ-6650

        Show
        jacques.le.roux Jacques Le Roux added a comment - Though unrelated (this bug report is anterior to the typo fixed by OFBIZ-6650 ) this has been fixed by OFBIZ-6650

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            jacques.le.roux Jacques Le Roux
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile