jenkinsci / publish-over-ftp-plugin

https://plugins.jenkins.io/publish-over-ftp/
21 stars 33 forks source link

[JENKINS-17165] "Publish over FTP" plugin with version 1.9 does'nt work after Jenkins upgrade to Version 1.504 #45

Closed jira-importer closed 6 years ago

jira-importer commented 11 years ago

I upgraded my Jenkins to Version 1.504, afer that "Publish over FTP" plugin with version 1.9 does'nt work. Suspecting Is it the issue after Jenkins upgrade or else issue is with the plugin.

Please kindly investigate on "publish over ftp" plug-in with version 1.9.

Thanks..


Originally reported by rkbejawar, imported from: "Publish over FTP" plugin with version 1.9 does'nt work after Jenkins upgrade to Version 1.504
  • assignee: bap
  • status: Resolved
  • priority: Major
  • resolution: Fixed
  • resolved: 2017-12-23T00:52:47+11:00
  • imported: 2022/01/10
jira-importer commented 11 years ago

bap:

It looks like the xstream upgrade in 1.504 has caused the issue. Until the compatability is resolved, you must use an earlier version of Jenkins

jira-importer commented 11 years ago

afischer:

I updated from Jenkins 1.497 to Jenkins 1.510. Now the only not working plugin is the "Publish over FTP"-plugin. The configuration is lost after restart of Jenkins.
How can we speedup the Bugfixing? Do you need some more support?

jira-importer commented 11 years ago

afischer:

I have resolved the problem for Publish-Over-FTP (I'm sure, the solution for Publish-Over-SSH is the same).
You have to override the method readResolve() in all inherited classes, for Publish-Over-FTP this are:

The reason is, that the reflection-mechanism looks for the method readResolve() in the inherited class (see above) and after this in the base-class. The reflection-invoke of the method fails, if the method is declared in the base-class and call over the derived class.
Can I check in the patched classes on GitHub?

jira-importer commented 11 years ago

afischer:

I have added a complete patched version of the plugins "publish-over" (base) and "publish-over-ftp" on GitHub:

Can you test it?

jira-importer commented 11 years ago

rameshpasham:

Hi Alexander,
I have installed the publish-over-ftp-plugin on jenkins 1.510 which is generated with your latest sources. It is working fine.
Thanks for fixing the issue.

jira-importer commented 11 years ago

bap:

Can everyone with this problem please provide their os, version including architecture and the full details of the JVM that the jenkins master is using?

jira-importer commented 11 years ago

rameshpasham:

HI bap,
Previously we had this issue but alexander's fix it got solved.
And for your requirement our jenkins-master info
os Red Hat Enterprise Linux Server release 6.2 (Santiago)
Kernel \r on an \m
java version "1.7.0"

jira-importer commented 11 years ago

afischer:

Hello bap,

I have the problem with our productive environment: Jenkins 1.510 under CentOS5.x (64bit) with Java 1.6.0_25 (64bit).
With a MS-Windows-testinstallation all seems to work with your old version, but the JUnit-tests have some problems...

With my patched version the plugin is working again under the productive environment and the unit-tests are ok under MS-Windows.

jira-importer commented 11 years ago

bap:

Thanks Ramesh, Alex,

Which JVM implementation are we talking about - Sun/ Open JDK / gcj ?

jira-importer commented 11 years ago

afischer:

We use the Sun/Oracle-Implementation.

jira-importer commented 11 years ago

rkbejawar:

Hi Alexander/bap, Thanks for the Fix.

jira-importer commented 11 years ago

denno:

When this Fix will be available as an official update?
CentOS 6.3, Jenkins 1.517

jira-importer commented 10 years ago

scm_issue_link:

Code changed in jenkins
User: Alexander Fischer
Path:
pom.xml
http://jenkins-ci.org/commit/publish-over-plugin/2e849359da4a1b9baf159ab7c888c564758a456d
Log:
fix for jenkins-bug #JENKINS-17165

exchange the version of JUnit to 4.9 (the same like in the other
modules)

jira-importer commented 10 years ago

scm_issue_link:

Code changed in jenkins
User: Alexander Fischer
Path:
src/main/java/jenkins/plugins/publish_over_ftp/BapFtpHostConfiguration.java
src/main/java/jenkins/plugins/publish_over_ftp/BapFtpPublisherPlugin.java
src/test/java/hudson/util/SecretHelper.java
http://jenkins-ci.org/commit/publish-over-ftp-plugin/aa3399f7c266a9d105d63e98dda654a34d110bc7
Log:
fix for jenkins-bug #JENKINS-17165

overriding method readResolve() from base-class BPHostConfiguration for
resolving Reflection-problems with XStream

jira-importer commented 10 years ago

scm_issue_link:

Code changed in jenkins
User: Alexander Fischer
Path:
pom.xml
http://jenkins-ci.org/commit/publish-over-ftp-plugin/33356f24624c5911f439afc6b0a411a9c2b6d21e
Log:
fix for jenkins-bug #JENKINS-17165

updated pom - integrate plugin-API for Jenkins1.510 and adapt for
Eclipse m2e

jira-importer commented 10 years ago

scm_issue_link:

Code changed in jenkins
User: Alexander Fischer
Path:
src/test/java/jenkins/plugins/publish_over_ftp/BapHostConfigurationTest.java
src/test/java/jenkins/plugins/publish_over_ftp/MySecretHelper.java
http://jenkins-ci.org/commit/publish-over-ftp-plugin/304aaaba58e3d130f3ec1e77e94ad2301452331f
Log:
fix for jenkins-bug #JENKINS-17165

overriding method readResolve() from base-class BPHostConfiguration for
resoving Reflection-problems with XStream;
removing own SecretHelper (replacement/hiding of the API-class) and
creating new class MySecretHelper;
resolves some problems with unit-tests under MS-Windows

jira-importer commented 6 years ago

slide_o_mix:

Looks like this was already resolved.

jira-importer commented 2 years ago

[Duplicates: JENKINS-17058

jira-importer commented 2 years ago

[Originally related to: JENKINS-12542]