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

NullPointerException accessing console

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • timestamper-plugin
    • None
    • jenkins deployed on tomcat 6

      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

          [JENKINS-16438] NullPointerException accessing console

          Nicolas De Loof created issue -

          Steven G Brown added a comment -

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

          Steven G Brown added a comment - Thanks for the well-researched bug report. Let me know if Timestamper 1.5.1 works for you.
          Steven G Brown made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          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

          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

          Steven G Brown added a comment -

          Does this work for you now?

          Steven G Brown added a comment - Does this work for you now?
          Steven G Brown made changes -
          Assignee Original: Steven G Brown [ stevengbrown ] New: Nicolas De Loof [ ndeloof ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 147258 ] New: JNJira + In-Review [ 192304 ]

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

              Created:
              Updated:
              Resolved: