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

NPE for completed project without a slack configuration

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • slack-plugin
    • None
    • Debian wheezy, Jenkins 1.580.1

      If the slack plugin is installed but a project does not have a slack configuration (which seems to happen with Matrix jobs) then an NPE is thrown and logged.

          [JENKINS-26066] NPE for completed project without a slack configuration

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: timp21337
          Path:
          src/main/java/jenkins/plugins/slack/ActiveNotifier.java
          http://jenkins-ci.org/commit/slack-plugin/d72d4b681e34339ba47123c264b1f28d181b34da
          Log:
          [FIXED JENKINS-26066] Add null check and log

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: timp21337 Path: src/main/java/jenkins/plugins/slack/ActiveNotifier.java http://jenkins-ci.org/commit/slack-plugin/d72d4b681e34339ba47123c264b1f28d181b34da Log: [FIXED JENKINS-26066] Add null check and log

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Sam Gleske
          Path:
          src/main/java/jenkins/plugins/slack/ActiveNotifier.java
          http://jenkins-ci.org/commit/slack-plugin/5ae2f2088e8db606ecbc2695a986a839d9faa72d
          Log:
          Merge PR #35 Add null check and log

          FIXED JENKINS-26066
          https://issues.jenkins-ci.org/browse/JENKINS-26066

          Compare: https://github.com/jenkinsci/slack-plugin/compare/cc82580a06fa...5ae2f2088e8d

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Sam Gleske Path: src/main/java/jenkins/plugins/slack/ActiveNotifier.java http://jenkins-ci.org/commit/slack-plugin/5ae2f2088e8db606ecbc2695a986a839d9faa72d Log: Merge PR #35 Add null check and log FIXED JENKINS-26066 https://issues.jenkins-ci.org/browse/JENKINS-26066 Compare: https://github.com/jenkinsci/slack-plugin/compare/cc82580a06fa...5ae2f2088e8d

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Sam Gleske
          Path:
          CHANGELOG.md
          README.markdown
          README.md
          pom.xml
          http://jenkins-ci.org/commit/slack-plugin/178b34abfc6340abefbd7ee7ecc6b95295bc5555
          Log:
          Release Slack 1.7.0

          New features:

          • Advanced settings
          • Team domain and token can be specified per project; falls back to
            global config if not specified (#19)
          • Post a list of commits with the build notification (title and author
            of commit) (#30, #44, #45)
          • Include JUnit test summary in build notifications (#25)
          • Use colors when sending a message using slack (#20, #24). Also, the
            start notifications use the color of the last build result (#43)
          • Support for authenticated proxies (#14)
          • Test Connection button (#46, #28)
          • Option to disable subsequent notifications on build failures (#46, #15)

          Improvements:

          • Report 'return to normal' on transition from unstable to success (#21)
          • Improved logging.

          Bug fixes:

          • When changing the global slack settings the slack config in jobs are
            updated as well (#26, #12)
          • Fix NullPointerException and output log message if slack is not
            configured (#35, JENKINS-26066)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Sam Gleske Path: CHANGELOG.md README.markdown README.md pom.xml http://jenkins-ci.org/commit/slack-plugin/178b34abfc6340abefbd7ee7ecc6b95295bc5555 Log: Release Slack 1.7.0 New features: Advanced settings Team domain and token can be specified per project; falls back to global config if not specified (#19) Post a list of commits with the build notification (title and author of commit) (#30, #44, #45) Include JUnit test summary in build notifications (#25) Use colors when sending a message using slack (#20, #24). Also, the start notifications use the color of the last build result (#43) Support for authenticated proxies (#14) Test Connection button (#46, #28) Option to disable subsequent notifications on build failures (#46, #15) Improvements: Report 'return to normal' on transition from unstable to success (#21) Improved logging. Bug fixes: When changing the global slack settings the slack config in jobs are updated as well (#26, #12) Fix NullPointerException and output log message if slack is not configured (#35, JENKINS-26066 )

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Sam Gleske
          Path:
          CHANGELOG.md
          README.markdown
          README.md
          pom.xml
          http://jenkins-ci.org/commit/slack-plugin/d6047ebe4344063178de1ef42304b69d453daf9c
          Log:
          Preparing for 1.7 release

          New features:

          • Advanced settings
          • Team domain and token can be specified per project; falls back to
            global config if not specified (#19)
          • Post a list of commits with the build notification (title and author
            of commit) (#30, #44, #45)
          • Include JUnit test summary in build notifications (#25)
          • Use colors when sending a message using slack (#20, #24). Also, the
            start notifications use the color of the last build result (#43)
          • Support for authenticated proxies (#14)
          • Test Connection button (#46, #28)
          • Option to disable subsequent notifications on build failures (#46, #15)

          Improvements:

          • Report 'return to normal' on transition from unstable to success (#21)
          • Improved logging.

          Bug fixes:

          • When changing the global slack settings the slack config in jobs are
            updated as well (#26, #12)
          • Fix NullPointerException and output log message if slack is not
            configured (#35, JENKINS-26066)

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Sam Gleske Path: CHANGELOG.md README.markdown README.md pom.xml http://jenkins-ci.org/commit/slack-plugin/d6047ebe4344063178de1ef42304b69d453daf9c Log: Preparing for 1.7 release New features: Advanced settings Team domain and token can be specified per project; falls back to global config if not specified (#19) Post a list of commits with the build notification (title and author of commit) (#30, #44, #45) Include JUnit test summary in build notifications (#25) Use colors when sending a message using slack (#20, #24). Also, the start notifications use the color of the last build result (#43) Support for authenticated proxies (#14) Test Connection button (#46, #28) Option to disable subsequent notifications on build failures (#46, #15) Improvements: Report 'return to normal' on transition from unstable to success (#21) Improved logging. Bug fixes: When changing the global slack settings the slack config in jobs are updated as well (#26, #12) Fix NullPointerException and output log message if slack is not configured (#35, JENKINS-26066 )

            sag47 Sam Gleske
            timp Tim Pizey
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: