Uploaded image for project: 'CXF'
  1. CXF
  2. CXF-8375

Access to httpprocessor field of HttpAsyncClientBuilder in AsyncHTTPConduitFactory

    XMLWordPrintableJSON

Details

    • New Feature
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 3.4.0
    • None
    • Transports
    • None
    • Unknown

    Description

      In CXF versions 2.7.18, we clearly had access to httpProcessor as per this commit https://github.com/apache/cxf/commit/a18be4fac03d3886317b0201123e9917d1a33e60#diff-f5349d24a3d1a3bced2aa168ce24d586461fe743d2e0b72f0fafcfedb9a99234L132.

      I don't know why access to that httpProcessor is revoked in the subsequent versions of CXF.

      We were facing a problem.  We were talking to a webservice.  The webservice returns a redirect 3XX response mentioning the new host to talk to.  And that host actually contains port as well such as "newserver.com:80".

      And while handling the redirect, CXF was putting the target host in the request as "newserver:80" instead of just "newserver".  This was failing and we were not able to fetch the results.

      The fix was to add a custom HTTP request interceptor in httpProcessor.  This interceptor stays on top of all the other interceptors.  And it simply chops the port and add the target host as just "newserver".

      It worked well in CXF version 2.7.18.  The moment we migrated to CXF 3.x (due to moving to JDK11) then our app starting giving compilation error.  Because in 3.x version, access to that httpProcessor is revoked.

      I would suggest to regain the access to that field since it was already like that before.  I know mine could be a rare requirement but it was easy for to add an interceptor to httpProcessor earlier.

      Right now, we have to create a custom factory and get the things done.  They are not that straight-forward.  If we had access to httpProcessor then no need to create any custom factory.  We can simply add our interceptors directly.

      Attachments

        Activity

          People

            Unassigned Unassigned
            syedbhai Syed Mudassir Ahmed
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: