Details

    • Bug
    • Status: Resolved (View Workflow)
    • Major
    • Resolution: Fixed
    • email-ext-plugin
    • None
    • jenkins 1.477
      email-ext 2.24.1

    Description

      For sender i have "${EMAIL_VAR} User@domain.com" and have NPE.
      Probably problem appeared after upgrading to 2.24.1 and token plugin...
      Email was triggered for: Success
      Email was triggered for: Fixed
      Trigger Success was overridden by another trigger and will not send an email.
      Sending email for trigger: Fixed
      ERROR: Could not send email as a part of the post-build publishers.
      java.lang.NullPointerException
      at java.util.regex.Matcher.getTextLength(Matcher.java:1140)
      at java.util.regex.Matcher.reset(Matcher.java:291)
      at java.util.regex.Matcher.<init>(Matcher.java:211)
      at java.util.regex.Pattern.matcher(Pattern.java:888)
      at hudson.plugins.emailext.plugins.ContentBuilder$Tokenizer.<init>(ContentBuilder.java:171)
      at hudson.plugins.emailext.plugins.ContentBuilder.replaceTokensWithContent(ContentBuilder.java:106)
      at hudson.plugins.emailext.plugins.ContentBuilder.replaceTokensWithContent(ContentBuilder.java:124)
      at hudson.plugins.emailext.plugins.ContentBuilder.transformText(ContentBuilder.java:88)
      at hudson.plugins.emailext.ExtendedEmailPublisher.getContent(ExtendedEmailPublisher.java:540)
      at hudson.plugins.emailext.ExtendedEmailPublisher.createMail(ExtendedEmailPublisher.java:363)
      at hudson.plugins.emailext.ExtendedEmailPublisher.sendMail(ExtendedEmailPublisher.java:273)
      at hudson.plugins.emailext.ExtendedEmailPublisher._perform(ExtendedEmailPublisher.java:265)
      at hudson.plugins.emailext.ExtendedEmailPublisher.perform(ExtendedEmailPublisher.java:225)
      at hudson.tasks.BuildStepMonitor$3.perform(BuildStepMonitor.java:36)
      at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:717)
      at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:692)
      at hudson.model.Build$BuildExecution.cleanUp(Build.java:192)
      at hudson.model.Run.execute(Run.java:1546)
      at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:46)
      at hudson.model.ResourceController.execute(ResourceController.java:88)
      at hudson.model.Executor.run(Executor.java:236)
      Finished: SUCCESS

      Attachments

        1. config.xml
          4 kB
          Kanstantsin Shautsou

        Activity

          I mean that i not sure what absolute path ${FILE} tried to load because my workspace path was strange.
          Yes, there shouldn't be NPE, if file doesn't exist. Btw, ${ENV} doesn't show anything in output if var=$VAR not exist.

          integer Kanstantsin Shautsou added a comment - I mean that i not sure what absolute path ${FILE} tried to load because my workspace path was strange. Yes, there shouldn't be NPE, if file doesn't exist. Btw, ${ENV} doesn't show anything in output if var=$VAR not exist.
          slide_o_mix Alex Earl added a comment -

          I'd rather put an error in so the user knows that the file was not found correctly in the path and what path they were trying to access.

          slide_o_mix Alex Earl added a comment - I'd rather put an error in so the user knows that the file was not found correctly in the path and what path they were trying to access.

          Yes, thanks. Will be tested on next release. Do i need to press resolved?

          integer Kanstantsin Shautsou added a comment - Yes, thanks. Will be tested on next release. Do i need to press resolved?
          slide_o_mix Alex Earl added a comment -

          Not until it is released and tested.

          slide_o_mix Alex Earl added a comment - Not until it is released and tested.
          slide_o_mix Alex Earl added a comment -

          Made the FILE token return an error if the file doesn't exist.

          slide_o_mix Alex Earl added a comment - Made the FILE token return an error if the file doesn't exist.

          People

            slide_o_mix Alex Earl
            integer Kanstantsin Shautsou
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: