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

NullPointerException accessing console

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: timestamper-plugin
    • Labels:
      None
    • Environment:
      jenkins deployed on tomcat 6
    • Similar Issues:

      Description

      ConsoleAnnotator Factory extracts Request.getPathInfo that is null

      from SRV.11.2 of the Servlet Spec (2.3), if the servlet mapping is "/" then the servletpath should be the uri minus context and pathinfo should
      be null.

      it seems tomcat is handling "/*" the same way as "/" (maybe a bug ?)
      anyway, a defensive null check and fallback to getServletpath would be fine

        Attachments

          Activity

          Hide
          stevengbrown Steven G Brown added a comment -

          Thanks for the well-researched bug report. Let me know if Timestamper 1.5.1 works for you.

          Show
          stevengbrown Steven G Brown added a comment - Thanks for the well-researched bug report. Let me know if Timestamper 1.5.1 works for you.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: StevenGBrown
          Path:
          src/main/java/hudson/plugins/timestamper/annotator/TimestampAnnotatorFactory.java
          http://jenkins-ci.org/commit/timestamper-plugin/9ca1cfa3a5b7144cbb009f261918bfa115b3301f
          Log:
          [FIXED JENKINS-16438] Fall back to servlet path when no path info

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: StevenGBrown Path: src/main/java/hudson/plugins/timestamper/annotator/TimestampAnnotatorFactory.java http://jenkins-ci.org/commit/timestamper-plugin/9ca1cfa3a5b7144cbb009f261918bfa115b3301f Log: [FIXED JENKINS-16438] Fall back to servlet path when no path info
          Hide
          stevengbrown Steven G Brown added a comment -

          Does this work for you now?

          Show
          stevengbrown Steven G Brown added a comment - Does this work for you now?

            People

            • Assignee:
              ndeloof Nicolas De Loof
              Reporter:
              ndeloof Nicolas De Loof
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: