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

Backslash broken on Properties Content injection

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Blocker
    • Resolution: Done
    • Component/s: envinject-plugin
    • Labels:
      None
    • Environment:
      EnvInject 1.66
      Jenkins 1.477
      Windows x64
    • Similar Issues:

      Description

      Inject environment variables to the build process > Properties Content,

      This used to work:

      Path=$Path;$WORKSPACE

      But now removes all \ from $Path and $WORKSPACE. Meaning Path looks like:

      Path = C:Windowssystem32;C:Windows;C:WindowsSystem32Wbem;C:WindowsSystem32WindowsPowerShellv1.0;...

        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/envinject/service/PropertiesLoader.java
          http://jenkins-ci.org/commit/envinject-plugin/92a497a905f1e70d0d664ac54e0fc9f7cb1a82da
          Log:
          Fix JENKINS-14761 and JENKINS-14768

          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/envinject/service/PropertiesLoader.java http://jenkins-ci.org/commit/envinject-plugin/92a497a905f1e70d0d664ac54e0fc9f7cb1a82da Log: Fix JENKINS-14761 and JENKINS-14768
          Hide
          jstarbird Jon Starbird added a comment - - edited

          This is broken again in 1.93.1 and 1.93.
          Properties like name=somevalue\someothervalue get imported as somevaluesomeothervalue

          Show
          jstarbird Jon Starbird added a comment - - edited This is broken again in 1.93.1 and 1.93. Properties like name=somevalue\someothervalue get imported as somevaluesomeothervalue
          Hide
          jstarbird Jon Starbird added a comment -

          Apologies, I just saw your old note about the changes for backslash handling. Somehow I missed the message about the changes.

          Show
          jstarbird Jon Starbird added a comment - Apologies, I just saw your old note about the changes for backslash handling. Somehow I missed the message about the changes.
          Hide
          jstarbird Jon Starbird added a comment -

          reopened in error. Closing again.

          Show
          jstarbird Jon Starbird added a comment - reopened in error. Closing again.

            People

            • Assignee:
              gbois Gregory Boissinot
              Reporter:
              wernight wernight
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: