GeoServer
  1. GeoServer
  2. GEOS-716

Put reverse proxy info in to GeoServer global controls

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.4.0-M1
    • Fix Version/s: 1.4.0-RC3
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      We have a fix to set geoserver running behind a reverse proxy, but it's just a variable in web.xml

      We should have it as an option in GeoServer. See: http://docs.codehaus.org/display/GEOSDOC/How+to+Config

      First step is to put as a global option and read and write the appropriate xml.

      Next is to put it in to the web admin tool. I'd say put it under Config -> Server.

      I know this will be painful, especially knowing wicket, but hopefully the experience will help you think more about how to simplify the design with wicket, and how to make the config system better in the future.

        Issue Links

          Activity

          Hide
          Andrea Aime added a comment -

          Fixed at revision
          The only thing I'd like to improve is to have validation of the parameter, but not sure how... should I try to connect to the provided url and complain if connection fails?

          Show
          Andrea Aime added a comment - Fixed at revision The only thing I'd like to improve is to have validation of the parameter, but not sure how... should I try to connect to the provided url and complain if connection fails?
          Hide
          Andrea Aime added a comment -

          Sorry, revision was 5280

          Show
          Andrea Aime added a comment - Sorry, revision was 5280
          Hide
          Andrea Aime added a comment -

          Well, it seems the fix wasn't working after all...

          Show
          Andrea Aime added a comment - Well, it seems the fix wasn't working after all...
          Hide
          Andrea Aime added a comment -

          Should be fixed both in 1.4.0rc3 and trunk

          Show
          Andrea Aime added a comment - Should be fixed both in 1.4.0rc3 and trunk
          Hide
          Andrea Aime added a comment -

          This issue has been resolved for one month or more without complaints. Closing it.

          Show
          Andrea Aime added a comment - This issue has been resolved for one month or more without complaints. Closing it.

            People

            • Assignee:
              Andrea Aime
              Reporter:
              Chris Holmes
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: