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

Add a checkbox called 'Expose P4TICKET' in environment

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Today, the plugin has a 'Expose P4PASSWD in environment' check box. This exposes both the P4PASSWD and P4TICKET environment variable.

      There are cases where I don't want to expose the P4PASSWD but I do want to use the P4TICKET. Can a 'Expose P4TICKET in environment' check box be added?

        Attachments

          Activity

          Hide
          rpetti Rob Petti added a comment -

          "Expose P4PASSWD" does not control the exposure of P4TICKET. P4TICKET is always exposed whenever it is available.

          – Posted from Bugbox for Android

          Show
          rpetti Rob Petti added a comment - "Expose P4PASSWD" does not control the exposure of P4TICKET. P4TICKET is always exposed whenever it is available. – Posted from Bugbox for Android
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Rob Petti
          Path:
          src/main/webapp/help/exposeP4Passwd.html
          http://jenkins-ci.org/commit/perforce-plugin/7f3da5ca2ee3145f3def82e2ba008c6bc5e4022c
          Log:
          JENKINS-14363 update online documentation so it matches current behavior

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Rob Petti Path: src/main/webapp/help/exposeP4Passwd.html http://jenkins-ci.org/commit/perforce-plugin/7f3da5ca2ee3145f3def82e2ba008c6bc5e4022c Log: JENKINS-14363 update online documentation so it matches current behavior
          Hide
          bsluis Ben Sluis added a comment -

          Ah, I shouldn't have assumed . Thanks for explaining!

          Show
          bsluis Ben Sluis added a comment - Ah, I shouldn't have assumed . Thanks for explaining!
          Hide
          rpetti Rob Petti added a comment -

          Yeah, the documentation was pretty misleading on that point. Do you think having an option to disable P4TICKET exposure would still be useful?

          Show
          rpetti Rob Petti added a comment - Yeah, the documentation was pretty misleading on that point. Do you think having an option to disable P4TICKET exposure would still be useful?
          Hide
          bsluis Ben Sluis added a comment -

          I don't think that would be necessary. There is no problem I see with exposure to the ticket. It doesn't expose user credentials or anything.

          Show
          bsluis Ben Sluis added a comment - I don't think that would be necessary. There is no problem I see with exposure to the ticket. It doesn't expose user credentials or anything.

            People

            • Assignee:
              rpetti Rob Petti
              Reporter:
              bsluis Ben Sluis
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: