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

Use the jenkins proxy configuration for the http protocol

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Minor Minor
    • notification-plugin
    • None

      The notification plugin does not take into account the proxy configuration configured in jenkins.
      It currently only adheres the http_proxy environment variable.

      I have a small pull request to take into account the proxy configuration of jenkins.

          [JENKINS-32270] Use the jenkins proxy configuration for the http protocol

          mcrooney added a comment - - edited

          Thanks for the PR! I've pinged one of the maintainers in it to see if it can be merged https://github.com/jenkinsci/notification-plugin/pull/24

          mcrooney added a comment - - edited Thanks for the PR! I've pinged one of the maintainers in it to see if it can be merged https://github.com/jenkinsci/notification-plugin/pull/24

          Code changed in jenkins
          User: Steven Aerts
          Path:
          src/main/java/com/tikal/hudson/plugins/notification/Protocol.java
          http://jenkins-ci.org/commit/notification-plugin/bd5709d93a032865bcc79d5d135d51b7d1618783
          Log:
          JENKINS-32270: use jenkins proxy configuration

          if no http_proxy environment variable is defined, use the jenkins
          proxy configuration if it is defined.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Steven Aerts Path: src/main/java/com/tikal/hudson/plugins/notification/Protocol.java http://jenkins-ci.org/commit/notification-plugin/bd5709d93a032865bcc79d5d135d51b7d1618783 Log: JENKINS-32270 : use jenkins proxy configuration if no http_proxy environment variable is defined, use the jenkins proxy configuration if it is defined.

          Code changed in jenkins
          User: Chen
          Path:
          src/main/java/com/tikal/hudson/plugins/notification/Protocol.java
          http://jenkins-ci.org/commit/notification-plugin/98ddf49139ebdd932ffc22b6e4bfd36c615576b1
          Log:
          Merge pull request #24 from steven-aerts/JENKINS-32270

          JENKINS-32270: use jenkins proxy configuration

          Compare: https://github.com/jenkinsci/notification-plugin/compare/df65c14599b5...98ddf49139eb

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Chen Path: src/main/java/com/tikal/hudson/plugins/notification/Protocol.java http://jenkins-ci.org/commit/notification-plugin/98ddf49139ebdd932ffc22b6e4bfd36c615576b1 Log: Merge pull request #24 from steven-aerts/ JENKINS-32270 JENKINS-32270 : use jenkins proxy configuration Compare: https://github.com/jenkinsci/notification-plugin/compare/df65c14599b5...98ddf49139eb

            Unassigned Unassigned
            steven_aerts Steven Aerts
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: