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

Avoid slave → master callables

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      See this discussion and this SECURITY-144 exemption for context. SubversionSCM.descriptor() and SubversionWorkspaceSelector.syncWorkspaceFormatFromMaster must not call Jenkins.getInstanceOrNull or similar. Checking JenkinsJVM is the quick way to determine if code is running on the master or an agent, but really this check should not be necessary at all. Rather, all information about master-side configuration should be prepared on the master and included in the master → slave callable's constructor and serializable state (CheckOutTask for example). PerJobCredentialStoreTest.remoteBuild is a key test.

      When fixed and released, the callable.conf entry can be deleted.

        Attachments

          Issue Links

            Activity

            jglick Jesse Glick created issue -
            jglick Jesse Glick made changes -
            Field Original Value New Value
            Link This issue relates to SECURITY-144 [ SECURITY-144 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-34857 [ JENKINS-34857 ]
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-48638 [ JENKINS-48638 ]
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/hudson/scm/SubversionSCM.java
            src/main/java/hudson/scm/SubversionWorkspaceSelector.java
            http://jenkins-ci.org/commit/subversion-plugin/3ab1af1e6f3e90e31267eaf52342ded6e753da15
            Log:
            Noting JENKINS-48543.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/hudson/scm/SubversionSCM.java src/main/java/hudson/scm/SubversionWorkspaceSelector.java http://jenkins-ci.org/commit/subversion-plugin/3ab1af1e6f3e90e31267eaf52342ded6e753da15 Log: Noting JENKINS-48543 .
            jglick Jesse Glick made changes -
            Link This issue relates to SECURITY-2456 [ SECURITY-2456 ]

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              jglick Jesse Glick
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated: