Karaf
  1. Karaf
  2. KARAF-690

install http feature will start both 8080 and 8181 jetty port

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.2, 3.0.0
    • Component/s: None
    • Labels:
      None

      Description

      jetty will start both 8080 and 8181 port if we install http feature, the configuration from pax-web can't override the one from jetty.xml.

      Simply remove those from our jetty.xml

      <!-- default port will be overwritten by pax-web configuration -->
      <Set name="port"><Property name="jetty.port" default="8080"/></Set>
      works and only let the configuration from pax-web take effect

        Activity

        Freeman Fang created issue -
        Freeman Fang made changes -
        Field Original Value New Value
        Assignee Freeman Fang [ ffang ]
        Freeman Fang made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Show
        Freeman Fang added a comment - commit fix http://svn.apache.org/viewvc?rev=1140484&view=rev for trunk http://svn.apache.org/viewvc?rev=1140486&view=rev for 2.2.x branch
        Freeman Fang made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 2.2.2 [ 12316339 ]
        Fix Version/s 3.0.0 [ 12316040 ]
        Resolution Fixed [ 1 ]
        Hide
        Jamie goodyear added a comment -

        Fixed in Apache Karaf 2.2.2.

        Show
        Jamie goodyear added a comment - Fixed in Apache Karaf 2.2.2.
        Jamie goodyear made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Piotr Wilkosz added a comment -

        The cleaner solution would be just to replace "Property" by" SystemProperty" in jetty.xml.
        This would also fix the the problem with controlling the jetty.host property from pax-web.

        • <Set name="host"><Property name="jetty.host" /></Set>
        • <Set name="port"><Property name="jetty.port" default="8080"/></Set>
          + <Set name="host"><SystemProperty name="jetty.host" /></Set>
          + <Set name="port"><SystemProperty name="jetty.port" default="8080"/></Set>
        Show
        Piotr Wilkosz added a comment - The cleaner solution would be just to replace "Property" by" SystemProperty" in jetty.xml. This would also fix the the problem with controlling the jetty.host property from pax-web. <Set name="host"><Property name="jetty.host" /></Set> <Set name="port"><Property name="jetty.port" default="8080"/></Set> + <Set name="host"><SystemProperty name="jetty.host" /></Set> + <Set name="port"><SystemProperty name="jetty.port" default="8080"/></Set>
        Hide
        Achim Nierbeck added a comment -

        Piotr Wilkosz Absolutely not.
        The only valid way is to use the osgi spec compliant properties, used by Pax Web, that's why those jetty.* properties have been removed from the jetty.xml.

        Show
        Achim Nierbeck added a comment - Piotr Wilkosz Absolutely not. The only valid way is to use the osgi spec compliant properties, used by Pax Web, that's why those jetty.* properties have been removed from the jetty.xml.

          People

          • Assignee:
            Freeman Fang
            Reporter:
            Freeman Fang
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development