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

NPE when using HockeyAppPlugin with HTTP Proxy

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: hockeyapp-plugin
    • Labels:
      None
    • Environment:
      Jenkins 2.34
      HockeyApp Plugin 1.2.2
    • Similar Issues:

      Description

      Hello,

      I'm seeing the following NPE with HockeyappPlugin (called via Jenkins Pipelines)

      16:57:53 java.lang.NullPointerException
      16:57:53 at java.util.concurrent.ConcurrentHashMap.putVal(ConcurrentHashMap.java:1011)
      16:57:53 at java.util.concurrent.ConcurrentHashMap.put(ConcurrentHashMap.java:1006)
      16:57:53 at org.apache.http.impl.client.BasicCredentialsProvider.setCredentials(BasicCredentialsProvider.java:61)
      16:57:53 at hockeyapp.HockeyappRecorder.createPreconfiguredHttpClient(HockeyappRecorder.java:165)
      16:57:53 at hockeyapp.HockeyappRecorder.performForApplication(HockeyappRecorder.java:252)
      16:57:53 at hockeyapp.HockeyappRecorder.perform(HockeyappRecorder.java:186)
      16:57:53 at org.jenkinsci.plugins.workflow.steps.CoreStep$Execution.run(CoreStep.java:69)
      16:57:53 at org.jenkinsci.plugins.workflow.steps.CoreStep$Execution.run(CoreStep.java:59)
      16:57:53 at org.jenkinsci.plugins.workflow.steps.AbstractSynchronousNonBlockingStepExecution$1$1.call(AbstractSynchronousNonBlockingStepExecution.java:52)
      16:57:53 at hudson.security.ACL.impersonate(ACL.java:221)
      16:57:53 at org.jenkinsci.plugins.workflow.steps.AbstractSynchronousNonBlockingStepExecution$1.run(AbstractSynchronousNonBlockingStepExecution.java:49)
      16:57:53 at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
      16:57:53 at java.util.concurrent.FutureTask.run(FutureTask.java:266)
      16:57:53 at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
      16:57:53 at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
      16:57:53 at java.lang.Thread.run(Thread.java:745)

      I'm using an HTTP Proxy which does not require a username or password.

      Thanks,
      Simon

        Attachments

          Activity

          Hide
          simonstjg Simon StJG added a comment -

          I've raise a pull request to fix this at https://github.com/jenkinsci/hockeyapp-plugin/pull/39

          Show
          simonstjg Simon StJG added a comment - I've raise a pull request to fix this at https://github.com/jenkinsci/hockeyapp-plugin/pull/39
          Hide
          rgroszewski Rafal Groszewski added a comment -

          Any plans to release that fix?

          Show
          rgroszewski Rafal Groszewski added a comment - Any plans to release that fix?
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Brantone
          Path:
          src/main/java/hockeyapp/HockeyappRecorder.java
          http://jenkins-ci.org/commit/hockeyapp-plugin/b4ace2f7b30b4e88dbf134eae81d18c0146214bd
          Log:
          Merge pull request #39 from SimonStJG/fix

          JENKINS-41873 Only set HTTP Proxy credentials provider if credentials are non-empty

          Compare: https://github.com/jenkinsci/hockeyapp-plugin/compare/920f70f66f18...b4ace2f7b30b

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Brantone Path: src/main/java/hockeyapp/HockeyappRecorder.java http://jenkins-ci.org/commit/hockeyapp-plugin/b4ace2f7b30b4e88dbf134eae81d18c0146214bd Log: Merge pull request #39 from SimonStJG/fix JENKINS-41873 Only set HTTP Proxy credentials provider if credentials are non-empty Compare: https://github.com/jenkinsci/hockeyapp-plugin/compare/920f70f66f18...b4ace2f7b30b
          Hide
          mezpahlan Mez Pahlan added a comment -

          Can we get an update on this please.

          Show
          mezpahlan Mez Pahlan added a comment - Can we get an update on this please.
          Hide
          mezpahlan Mez Pahlan added a comment -

          Am I correct in thinking that a full fix also requires https://github.com/jenkinsci/hockeyapp-plugin/pull/47 to be merged?

          Show
          mezpahlan Mez Pahlan added a comment - Am I correct in thinking that a full fix also requires https://github.com/jenkinsci/hockeyapp-plugin/pull/47  to be merged?
          Hide
          mezpahlan Mez Pahlan added a comment -

          Closing this as both PR39 and PR47 have been merged into master.

          Show
          mezpahlan Mez Pahlan added a comment - Closing this as both PR39 and PR47 have been merged into master.

            People

            • Assignee:
              Unassigned
              Reporter:
              simonstjg Simon StJG
            • Votes:
              2 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: