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

URLTrigger: Invalid configurations

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      After various option permutations, I finally was able to inspect a text response with:

      • Inspect URL content
      • Monitor a change of the content
        I couldn't get it to work with any other combination except only and exactly those two.

      I found the interface very confusing...
      1. Checking "[URLTrigger] - Poll with a URL" w/o specifying anything else polls, but never finds any changes. Users should be notified that this is an invalid state, in the config, output, or both
      2. [1] and checking "Inspect URL content" causes an error (see [4] below)
      3. TXT/"Monitor the contents of a text response" seems especially misleading, since it requires a regex. Not knowing that (although I now see it in the help), I set TXT w/o a regex only to discover later that it was never finding any changes in the context. I think either:

      • if no regex is supplied, assume all text (.*) - I like this one the best
      • notify the user that TXT w/o a regex is an invalid config
      • name it REGEX/"Something mentioning regex"

      [4] Error output:
      Error

      Failed to parse form data. Please report this problem as a bug
      JSON={"":"","builder":

      {"command":"\"$JENKINS_HOME\"/build.sh","kind":"hudson.tasks.Shell","stapler-class":"hudson.tasks.Shell"}

      ,"core:apply":"","description":"","displayNameOrNull":"","logrotate":

      {"logrotate_artifact_days":"","logrotate_artifact_nums":"","logrotate_days":"","logrotate_nums":"5"}

      ,"name":"Test Job","org-jenkinsci-plugins-urltrigger-URLTrigger":{"cronTabSpec":"* * * * *","urlElements":{"checkETag":false,"checkLastModificationDate":false,"inspectingContent":{},"password":"","proxyActivated":false,"url":"http://seandenigris.com/blog/?page_id=58","username":""}},"properties":{"hudson-model-ParametersDefinitionProperty":{},"stapler-class-bag":"true"},"scm":{"value":"1"}}

      net.sf.json.JSONException: null object
      at net.sf.json.JSONObject.verifyIsNull(JSONObject.java:2428)

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Gregory Boissinot
          Path:
          src/main/java/org/jenkinsci/plugins/urltrigger/URLTrigger.java
          src/main/java/org/jenkinsci/plugins/urltrigger/content/JSONContentType.java
          src/main/java/org/jenkinsci/plugins/urltrigger/content/TEXTContentEntry.java
          src/main/java/org/jenkinsci/plugins/urltrigger/content/TEXTContentType.java
          src/main/java/org/jenkinsci/plugins/urltrigger/content/XMLContentType.java
          src/main/resources/org/jenkinsci/plugins/urltrigger/content/TEXTContentType/config.jelly
          src/main/resources/org/jenkinsci/plugins/urltrigger/content/TEXTContentType/help-regEx.html
          http://jenkins-ci.org/commit/urltrigger-plugin/a7888eede69ab42f1c0bbe3b4a1358ade354213f
          Log:
          Fix JENKINS-14620

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gregory Boissinot Path: src/main/java/org/jenkinsci/plugins/urltrigger/URLTrigger.java src/main/java/org/jenkinsci/plugins/urltrigger/content/JSONContentType.java src/main/java/org/jenkinsci/plugins/urltrigger/content/TEXTContentEntry.java src/main/java/org/jenkinsci/plugins/urltrigger/content/TEXTContentType.java src/main/java/org/jenkinsci/plugins/urltrigger/content/XMLContentType.java src/main/resources/org/jenkinsci/plugins/urltrigger/content/TEXTContentType/config.jelly src/main/resources/org/jenkinsci/plugins/urltrigger/content/TEXTContentType/help-regEx.html http://jenkins-ci.org/commit/urltrigger-plugin/a7888eede69ab42f1c0bbe3b4a1358ade354213f Log: Fix JENKINS-14620
          Hide
          gbois Gregory Boissinot added a comment -

          I have beginning to fix the issue.
          Please could you check if it suits you before I deliver a new release?

          Show
          gbois Gregory Boissinot added a comment - I have beginning to fix the issue. Please could you check if it suits you before I deliver a new release?
          Hide
          gbois Gregory Boissinot added a comment -

          Without any response, I made a release.
          Feel free to reopen the issue or raise an new issue if it the solution doesn't suit you.
          Thanks

          Show
          gbois Gregory Boissinot added a comment - Without any response, I made a release. Feel free to reopen the issue or raise an new issue if it the solution doesn't suit you. Thanks

            People

            Assignee:
            gbois Gregory Boissinot
            Reporter:
            seandenigris Sean DeNigris
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: