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

EnvInject is messing around with my TEMP variable when username contains a dollar sign

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Blocker Blocker
    • envinject-plugin
    • None
    • EnvInject 1.46

      My Jenkins slaves are using users starting with a dollar sign.

      The plugin seems to handle dollar sign with a special case because it is breaking my TEMP variable

      The normal value is C:\DOCUME~1\$NCESE~1\LOCALS~1\Temp
      But when using the plugin, it changes it to C:\DOCUME~1$NCESE~1\LOCALS~1\Temp (notice the removed \ before the $)

          [JENKINS-13566] EnvInject is messing around with my TEMP variable when username contains a dollar sign

          Code changed in jenkins
          User: Gregory Boissinot
          Path:
          src/main/java/org/jenkinsci/plugins/envinject/service/EnvInjectEnvVars.java
          http://jenkins-ci.org/commit/envinject-plugin/9f43a3a2ee2312c763c73e10281f4b1fc34f9106
          Log:
          Fix JENKINS-13566

          Compare: https://github.com/jenkinsci/envinject-plugin/compare/8645f86...9f43a3a

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gregory Boissinot Path: src/main/java/org/jenkinsci/plugins/envinject/service/EnvInjectEnvVars.java http://jenkins-ci.org/commit/envinject-plugin/9f43a3a2ee2312c763c73e10281f4b1fc34f9106 Log: Fix JENKINS-13566 Compare: https://github.com/jenkinsci/envinject-plugin/compare/8645f86...9f43a3a

          Gregory Boissinot added a comment - - edited

          @Vincent
          Please could you give a comment to the issue JENKINS-15751?
          What are your opinion?
          Are your ready to double your slash (back from previous state), any strong impact for you?

          Gregory Boissinot added a comment - - edited @Vincent Please could you give a comment to the issue JENKINS-15751 ? What are your opinion? Are your ready to double your slash (back from previous state), any strong impact for you?

            gbois Gregory Boissinot
            vlatombe Vincent Latombe
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: