Uploaded image for project: 'ServiceMix'
  1. ServiceMix
  2. SM-780

Bug due to change in proxy support for http binding component

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.1
    • 3.1
    • servicemix-http
    • None
    • All

    • Patch Available

    Description

      The class ProviderProcessor supports proxies now. However, there is a NullPointer Exception, when no proxy is configured. Please verify the attached diff.

      Best regards.
      Martin

      Index: D:/JBossIDE/ServiceMix/ServiceMix-SNAPSHOT/deployables/bindingcomponents/servicemix-http/src/main/java/org/apache/servicemix/http/processors/ProviderProcessor.java
      ===================================================================
      — D:/JBossIDE/ServiceMix/ServiceMix-SNAPSHOT/deployables/bindingcomponents/servicemix-http/src/main/java/org/apache/servicemix/http/processors/ProviderProcessor.java (revision 487484)
      +++ D:/JBossIDE/ServiceMix/ServiceMix-SNAPSHOT/deployables/bindingcomponents/servicemix-http/src/main/java/org/apache/servicemix/http/processors/ProviderProcessor.java (working copy)
      @@ -260,12 +260,12 @@
      if ((endpoint.getProxy().getProxyHost() != null) && (endpoint.getProxy().getProxyPort() != 0))

      { host.setProxy(endpoint.getProxy().getProxyHost(), endpoint.getProxy().getProxyPort()); }

      + if (endpoint.getProxy().getProxyCredentials() != null)

      { + endpoint.getProxy().getProxyCredentials().applyProxyCredentials(getClient()); + }

      } else if ((getConfiguration().getProxyHost() != null) && (getConfiguration().getProxyPort() != 0))

      { host.setProxy(getConfiguration().getProxyHost(), getConfiguration().getProxyPort()); }
      • if (endpoint.getProxy().getProxyCredentials() != null) { - endpoint.getProxy().getProxyCredentials().applyProxyCredentials(getClient()); - }

        return host;
        }

      Attachments

        Activity

          People

            Unassigned Unassigned
            mlandua Martin Landua
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: