Uploaded image for project: 'Apache Knox'
  1. Apache Knox
  2. KNOX-1559

Create Dispatch implementation that is configurable via service.xml file

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

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.3.0
    • None

    Description

      Create a org.apache.knox.gateway.dispatch.Dispatch implementation that is configurable via metadata file (for example, service.xml). Configurable parameters should be

      • headers to exclude when forwarding requests
      • whether parameters should URL-encoded or not

      Such an implementation should be able to make at least the following Dispatch implementations obsolete:

      • org.apache.knox.gateway.dispatch.PassAllHeadersNoEncodingDispatch
      • org.apache.knox.gateway.dispatch.PassAllHeadersDispatch

      A possible declaration could be:

          <dispatch classname="org.apache.knox.gateway.dispatch.ConfigurableDispatch">
             <property>
                 <name>excludeHeaders<name>
                 <value>Authorization,Content-Length</value>
             </property>
             <property>
                 <name>removeUrlEncoding<name>
                 <value>false</value>
             </property>
          </dispatch>
      

      Or maybe

          <dispatch classname="org.apache.knox.gateway.dispatch.ConfigurableDispatch" removeUrlEncoding="true" excludeHeaders="Authorization, Content-Length" />
      

      Attachments

        1. KNOX-1559.patch
          17 kB
          Kevin Risden
        2. KNOX-1559.patch
          17 kB
          Kevin Risden
        3. KNOX-1559.patch
          41 kB
          Kevin Risden
        4. KNOX-1559.patch
          26 kB
          Kevin Risden
        5. KNOX-1559.patch
          26 kB
          Kevin Risden
        6. KNOX-1559.patch
          26 kB
          Kevin Risden

        Issue Links

        Activity

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

          People

            krisden Kevin Risden
            rlevas Robert Levas
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment