Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-28289

JNLP slave should exclude non proxy hosts when connecting via HTTP proxy

      https://issues.jenkins-ci.org/browse/JENKINS-6167
      The fix submitted for this issue does not take into account the excluded proxy hosts unlike the hudson.remoting.Util.openURLConnection method which implicitly excludes hosts.

      A fix using the same method as sun.net.www.protocol.http.HttpURLConnection is available here :
      https://github.com/hypnoce/remoting/commit/67dbd966e628c42a0aa4d3cd91a56a00b7f95030

      Thanks

          [JENKINS-28289] JNLP slave should exclude non proxy hosts when connecting via HTTP proxy

          Francis JAC created issue -
          James Nord made changes -
          Remote Link New: This issue links to "PR#55 (Web Link)" [ 13144 ]
          Evan Wee made changes -
          Priority Original: Minor [ 4 ] New: Major [ 3 ]
          Evan Wee made changes -
          Description Original: https://issues.jenkins-ci.org/browse/JENKINS-6167
          The fix submitted for this issue does not take into account the excluded proxy hosts unlike the hudson.remoting.Util.openURLConnection method which implicitly excludes hosts.

          A fix using the same method as sun.net.www.protocol.http.HttpURLConnection is available here :
          https://github.com/hypnoce/remoting/commit/67dbd966e628c42a0aa4d3cd91a56a00b7f95030

          Thanks
          New: https://issues.jenkins-ci.org/browse/JENKINS-6167
          The fix submitted for this issue does not take into account the excluded proxy hosts unlike the hudson.remoting.Util.openURLConnection method which implicitly excludes hosts.

          A fix using the same method as sun.net.www.protocol.http.HttpURLConnection is available here :
          https://github.com/hypnoce/remoting/commit/67dbd966e628c42a0aa4d3cd91a56a00b7f95030

          Thanks

          EDIT:
          An updated fix that resolves conflicts is here:
          https://github.com/jenkinsci/remoting/pull/62
          https://github.com/evanx9/remoting/commit/cb2ded8e8ba6a67dd5996a7ca90a2f4f133ec8eb
          Evan Wee made changes -
          Remote Link New: This issue links to "PR#62 (Web Link)" [ 13170 ]
          Daniel Beck made changes -
          Labels New: lts-candidate
          Daniel Beck made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Oliver Gondža made changes -
          Labels Original: lts-candidate New: 1.625.3-rejected lts-candidate
          Evan Wee made changes -
          Remote Link Original: This issue links to "PR#62 (Web Link)" [ 13170 ]
          Evan Wee made changes -
          Description Original: https://issues.jenkins-ci.org/browse/JENKINS-6167
          The fix submitted for this issue does not take into account the excluded proxy hosts unlike the hudson.remoting.Util.openURLConnection method which implicitly excludes hosts.

          A fix using the same method as sun.net.www.protocol.http.HttpURLConnection is available here :
          https://github.com/hypnoce/remoting/commit/67dbd966e628c42a0aa4d3cd91a56a00b7f95030

          Thanks

          EDIT:
          An updated fix that resolves conflicts is here:
          https://github.com/jenkinsci/remoting/pull/62
          https://github.com/evanx9/remoting/commit/cb2ded8e8ba6a67dd5996a7ca90a2f4f133ec8eb
          New: https://issues.jenkins-ci.org/browse/JENKINS-6167
          The fix submitted for this issue does not take into account the excluded proxy hosts unlike the hudson.remoting.Util.openURLConnection method which implicitly excludes hosts.

          A fix using the same method as sun.net.www.protocol.http.HttpURLConnection is available here :
          https://github.com/hypnoce/remoting/commit/67dbd966e628c42a0aa4d3cd91a56a00b7f95030

          Thanks

          Oliver Gondža made changes -
          Labels Original: 1.625.3-rejected lts-candidate New: 1.625.3-rejected 1.642.2-rejected lts-candidate

            Unassigned Unassigned
            hypnoce Francis JAC
            Votes:
            0 Vote for this issue
            Watchers:
            10 Start watching this issue

              Created:
              Updated:
              Resolved: