• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • None
    • Jenkins 1.646
      Poll Mailbox Trigger Plugin 1.020

      Overriding subjectContains in Advanced Email Properties has no effect.
      See attached Poll Mailbox Trigger Log

          [JENKINS-32842] Cannot override subjectContains

          jako512 added a comment - - edited

          I can confirm that problem is related to version 1.020:

          jako512 added a comment - - edited I can confirm that problem is related to version 1.020:

          jako512 added a comment -

          Now I have reproduced the problem on clean docker image with only this plugin installed.

          jako512 added a comment - Now I have reproduced the problem on clean docker image with only this plugin installed.

          kubhera a added a comment - - edited

          Hi, even i am facing the same issue with this plugin.. not able to override the subject string

          kubhera a added a comment - - edited Hi, even i am facing the same issue with this plugin.. not able to override the subject string

          Nick Grealy added a comment -

          Just an update:

          I'm just following this up with the previous committer here -> https://github.com/jenkinsci/poll-mailbox-trigger-plugin/pull/1 .

          I'll have some time to look at this issue tomorrow, and will hopefully have a new release pushed out by the weekend.

          Thanks,
          Nick

          Nick Grealy added a comment - Just an update: I'm just following this up with the previous committer here -> https://github.com/jenkinsci/poll-mailbox-trigger-plugin/pull/1 . I'll have some time to look at this issue tomorrow, and will hopefully have a new release pushed out by the weekend. Thanks, Nick

          Code changed in jenkins
          User: Nick Grealy
          Path:
          build.gradle
          dependencies.txt
          gradle.properties
          gradle/common.gradle
          src/main/groovy/org/jenkinsci/plugins/pollmailboxtrigger/PollMailboxTrigger.java
          src/main/groovy/org/jenkinsci/plugins/pollmailboxtrigger/SafeJenkins.java
          src/main/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/utils/CustomProperties.java
          src/main/resources/org/jenkinsci/plugins/pollmailboxtrigger/PollMailboxTrigger/config.jelly
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/AcceptanceTests.java
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/StepDefs.groovy
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/WipAcceptanceTests.java
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/bdd/ConfigurationRow.groovy
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/bdd/EmailRow.groovy
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/PollMailboxTrigger_TestConnectionTest.groovy
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/QuickTest.groovy
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/testingTools/MailboxIntegrationTest.groovy
          src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/testingTools/MessageBuilder.groovy
          src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Configuration.feature
          src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Runtime Configuration.feature
          src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Test Connection Errors.feature
          src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Test Connection Success.feature
          http://jenkins-ci.org/commit/poll-mailbox-trigger-plugin/b9fd9265d5e22a8492fc10f34d7f16530ca7c76a
          Log:
          Fixing issues JENKINS-32842 , JENKINS-33136

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nick Grealy Path: build.gradle dependencies.txt gradle.properties gradle/common.gradle src/main/groovy/org/jenkinsci/plugins/pollmailboxtrigger/PollMailboxTrigger.java src/main/groovy/org/jenkinsci/plugins/pollmailboxtrigger/SafeJenkins.java src/main/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/utils/CustomProperties.java src/main/resources/org/jenkinsci/plugins/pollmailboxtrigger/PollMailboxTrigger/config.jelly src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/AcceptanceTests.java src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/StepDefs.groovy src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/WipAcceptanceTests.java src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/bdd/ConfigurationRow.groovy src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/bdd/EmailRow.groovy src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/PollMailboxTrigger_TestConnectionTest.groovy src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/QuickTest.groovy src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/testingTools/MailboxIntegrationTest.groovy src/test/groovy/org/jenkinsci/plugins/pollmailboxtrigger/mail/testingTools/MessageBuilder.groovy src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Configuration.feature src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Runtime Configuration.feature src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Test Connection Errors.feature src/test/resources/org/jenkinsci/plugins/pollmailboxtrigger/Test Connection Success.feature http://jenkins-ci.org/commit/poll-mailbox-trigger-plugin/b9fd9265d5e22a8492fc10f34d7f16530ca7c76a Log: Fixing issues JENKINS-32842 , JENKINS-33136

          Nick Grealy added a comment -

          I've just pushed a new release - 1.021 - it should be available in the next 24hrs.

          Can you please retest this issue with the new version, to confirm the problem has been fixed?

          Kind regards,
          Nick Grealy

          Nick Grealy added a comment - I've just pushed a new release - 1.021 - it should be available in the next 24hrs. Can you please retest this issue with the new version, to confirm the problem has been fixed? Kind regards, Nick Grealy

          jako512 added a comment -

          I can confirm it is working now.
          Thank You !

          jako512 added a comment - I can confirm it is working now. Thank You !

          Nick Grealy added a comment -

          Not a problem. Sorry for introducing the issue, and thanks for letting me know about it!

          Nick Grealy added a comment - Not a problem. Sorry for introducing the issue, and thanks for letting me know about it!

          kubhera a added a comment -

          Tested and it is working fine now!!!

          Thanks a lot Nick

          kubhera a added a comment - Tested and it is working fine now!!! Thanks a lot Nick

          Nick Grealy added a comment -

          A pleasure, thanks for the feedback!

          Nick Grealy added a comment - A pleasure, thanks for the feedback!

            nickg Nick Grealy
            jako512 jako512
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: