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

The Solr redirection does not work in trunk demo

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: solr
    • Labels:
      None

      Activity

      Hide
      shi.jinghai Shi Jinghai added a comment -

      I submitted some changes to trunk rev. 1700739:

      1. Use response.sendRedirect to forward url to login page instead of the response.setHeader("location", url), this avoids the warning messages from EntityUtilProperties. Besides, when testing in similar config as demo-ofbiz-trunk is (only 80 and 443 open, port 80 -> 8080, port 443 -> 8443), this change can keep the url to 80 or 443.

      2. Replace tabs to spaces in several java files.

      Hope it works on demo-ofbiz-trunk.

      Show
      shi.jinghai Shi Jinghai added a comment - I submitted some changes to trunk rev. 1700739: 1. Use response.sendRedirect to forward url to login page instead of the response.setHeader("location", url), this avoids the warning messages from EntityUtilProperties. Besides, when testing in similar config as demo-ofbiz-trunk is (only 80 and 443 open, port 80 -> 8080, port 443 -> 8443), this change can keep the url to 80 or 443. 2. Replace tabs to spaces in several java files. Hope it works on demo-ofbiz-trunk.
      Hide
      shi.jinghai Shi Jinghai added a comment -

      Just tested locally, not work well.

      Show
      shi.jinghai Shi Jinghai added a comment - Just tested locally, not work well.
      Hide
      shi.jinghai Shi Jinghai added a comment -

      I submitted a trunk rev. 1701379. Scheme https check is removed, and http://demo-trunk-ofbiz.apache.org/solr/ can be displayed correctly now.

      Show
      shi.jinghai Shi Jinghai added a comment - I submitted a trunk rev. 1701379. Scheme https check is removed, and http://demo-trunk-ofbiz.apache.org/solr/ can be displayed correctly now.
      Hide
      jacques.le.roux Jacques Le Roux added a comment -

      Thanks Jinghai!

      Show
      jacques.le.roux Jacques Le Roux added a comment - Thanks Jinghai!

        People

        • Assignee:
          shi.jinghai Shi Jinghai
          Reporter:
          jacques.le.roux Jacques Le Roux
        • Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development