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

16.11 ofbizUrl include host+port and break some reverse-proxy / docker setups

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • Release Branch 16.11
    • 16.11.02
    • None
    • None

    Description

      OFBiz 16.11 has changed how it generates urls in its templates.
      It sometimes includes hostname+port.

      This can be reproduced on the official ofbiz-demo site:

      Go to http://ofbiz.apache.org/

      • Open the Ofbiz 13.07 Demo (Management Apps)
      • - it will auto-login for your, do a logout so that you can see the login-mask
      • - look at the HTML code and look for the form-post url etc. You can see that none of the URLs include hostname:port. They are all "/url/path"
      • Open the Ofbiz 16.11 Demo (Management Apps)
      • - it will auto-login for your, do a logout so that you can see the login-mask
      • - look at the HTML code and look for the form-post url etc.
      • - the post-urls and login-links have format: https://hostname:port/url/path

      The new 16.11 behaviour can cause problems in cluster / reverse-proxy setups where actual host:port does not match public host:port

      Before 16.11 this was not an issue because all links were without hostname:port. But now the login masks loads but the subsequent links all show the wrong port.

      url.properties does control port/hostname but preferably we wouldn't need to configure it.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            jleroux Jacques Le Roux
            leonardlin Leonard Lin
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment