Skip to content

Commit

Permalink
Migrate from EE 8 to EE 9
Browse files Browse the repository at this point in the history
  • Loading branch information
basil committed Jan 22, 2025
1 parent 5dfecc3 commit f0784a9
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 12 deletions.
8 changes: 4 additions & 4 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -28,16 +28,16 @@
<parent>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>plugin</artifactId>
<version>4.74</version>
<version>5.5</version>
<relativePath />
</parent>

<properties>
<changelist>999999-SNAPSHOT</changelist>
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
<!-- https://www.jenkins.io/doc/developer/plugin-development/choosing-jenkins-baseline/ -->
<jenkins.baseline>2.361</jenkins.baseline>
<jenkins.version>${jenkins.baseline}.4</jenkins.version>
<jenkins.baseline>2.479</jenkins.baseline>
<jenkins.version>${jenkins.baseline}.1</jenkins.version>
</properties>

<artifactId>publish-over-ssh</artifactId>
Expand Down Expand Up @@ -69,7 +69,7 @@
<dependency>
<groupId>io.jenkins.tools.bom</groupId>
<artifactId>bom-${jenkins.baseline}.x</artifactId>
<version>2102.v854b_fec19c92</version>
<version>3850.vb_c5319efa_e29</version>
<scope>import</scope>
<type>pom</type>
</dependency>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import com.fasterxml.jackson.core.type.TypeReference;
import com.fasterxml.jackson.databind.ObjectMapper;
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import hudson.FilePath;
import hudson.remoting.VirtualChannel;
import org.jenkinsci.remoting.RoleChecker;
Expand Down Expand Up @@ -124,6 +125,7 @@ public BapSshTransferCacheRow() {}
*
* @param fileRef
*/
@SuppressFBWarnings(value = "PA_PUBLIC_PRIMITIVE_ATTRIBUTE", justification = "Backwards compatibility")
public BapSshTransferCacheRow(File fileRef) {
HashValue = getHashValue(fileRef);
try {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ public FormValidation doCheckUsername(@QueryParameter final String value) {
}

public FormValidation doCheckKeyPath(@QueryParameter final String value) {
AccessControlled subject = Stapler.getCurrentRequest().findAncestorObject(AbstractProject.class);
AccessControlled subject = Stapler.getCurrentRequest2().findAncestorObject(AbstractProject.class);

Check warning on line 61 in src/main/java/jenkins/plugins/publish_over_ssh/descriptor/BapSshCredentialsDescriptor.java

View check run for this annotation

ci.jenkins.io / Code Coverage

Not covered line

Line 61 is not covered by tests
if (subject == null) {
subject = Jenkins.getInstance();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@
import jenkins.plugins.publish_over_ssh.BapSshPublisherPlugin;
import jenkins.plugins.publish_over_ssh.Messages;
import org.kohsuke.stapler.QueryParameter;
import org.kohsuke.stapler.StaplerRequest;
import org.kohsuke.stapler.StaplerResponse;
import org.kohsuke.stapler.StaplerRequest2;
import org.kohsuke.stapler.StaplerResponse2;
import org.kohsuke.stapler.interceptor.RequirePOST;

@Extension
Expand Down Expand Up @@ -92,7 +92,7 @@ public FormValidation doCheckKeyPath(@QueryParameter final String value) {
}

@RequirePOST
public FormValidation doTestConnection(final StaplerRequest request, final StaplerResponse response) {
public FormValidation doTestConnection(final StaplerRequest2 request, final StaplerResponse2 response) {
Jenkins.get().checkPermission(Jenkins.ADMINISTER);

final BapSshPublisherPlugin.Descriptor pluginDescriptor;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@
import hudson.util.Secret;
import jenkins.plugins.publish_over.BPHostConfiguration;
import org.apache.commons.lang.StringUtils;
import org.kohsuke.stapler.StaplerRequest;
import org.kohsuke.stapler.StaplerResponse;
import org.kohsuke.stapler.StaplerRequest2;
import org.kohsuke.stapler.StaplerResponse2;

import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import hudson.Util;
Expand Down Expand Up @@ -145,7 +145,7 @@ public void removeHostConfiguration(final String name) {
}

@Override
public boolean configure(final StaplerRequest request, final JSONObject formData) {
public boolean configure(final StaplerRequest2 request, final JSONObject formData) {
final List<BapSshHostConfiguration> newConfigurations = request.bindJSONToList(BapSshHostConfiguration.class,
formData.get("instance"));
commonConfig = request.bindJSON(BapSshCommonConfiguration.class, formData.getJSONObject("commonConfig"));
Expand Down Expand Up @@ -192,7 +192,7 @@ public jenkins.plugins.publish_over.view_defaults.manage_jenkins.Messages getCom
}

@RequirePOST
public FormValidation doTestConnection(final StaplerRequest request, final StaplerResponse response) {
public FormValidation doTestConnection(final StaplerRequest2 request, final StaplerResponse2 response) {
Jenkins.get().checkPermission(Jenkins.ADMINISTER);
final BapSshHostConfiguration hostConfig = request.bindParameters(BapSshHostConfiguration.class, "");
hostConfig.setCommonConfig(request.bindParameters(BapSshCommonConfiguration.class, "common."));
Expand Down

0 comments on commit f0784a9

Please sign in to comment.