Skip to content

[JENKINS-69868] Avoid deserialization of anonymous classes - #61

Open
chriskloe wants to merge 12 commits into
jenkinsci:masterfrom
chriskloe:master
Open

[JENKINS-69868] Avoid deserialization of anonymous classes#61
chriskloe wants to merge 12 commits into
jenkinsci:masterfrom
chriskloe:master

Conversation

@chriskloe

Copy link
Copy Markdown
Contributor

Avoid deserialization of anonymous classes. Found a hint in the console log and followed the advice: https://www.jenkins.io/doc/developer/extensibility/serialization-of-anonymous-classes/

Affected issue: JENKINS-69868

Avoid deserialization of anonymous classes. Found a hint in the console log and followed the advice: https://www.jenkins.io/doc/developer/extensibility/serialization-of-anonymous-classes/
@MarkEWaite MarkEWaite changed the title Update AbstractCvs.java Avoid deserialization of anonymous classes Oct 15, 2022
@MarkEWaite MarkEWaite changed the title Avoid deserialization of anonymous classes [JENKINS-69868] Avoid deserialization of anonymous classes Oct 15, 2022
@MarkEWaite MarkEWaite added chore Reduces maintenance effort by changes not directly visible to users hacktoberfest-accepted labels Oct 15, 2022

@gounthar gounthar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.
Thanks.

@gounthar

Copy link
Copy Markdown
Contributor

The failure is due to a network or machine connection problem, not to the code itself.

@MarkEWaite

Copy link
Copy Markdown
Contributor

Thanks for the contribution @chriskloe . Apologies that it took so long to merge.

@chriskloe

Copy link
Copy Markdown
Contributor Author

No need to apologize, there has not been a release for some time. Even my first fix is not released yet. On my server I'm working with a locally built version. Maybe enabling incremental builds will help here.
At the moment it looks like I am creating a lot confusion with my commits. I am not yet used to git, github and the git plugin in eclipse.
Today's commits were not intended to go to this PR but instead to individual new ones.
I'm very sorry for that!

@MarkEWaite

Copy link
Copy Markdown
Contributor

@chriskloe the CVS plugin could really use a new maintainer. Are you still using the plugin? If so, are you interested in adopting the plugin?

Active users of a plugin are often the best maintainers, since they are most likely to release new versions as needed and are most likely to merge pull requests and verify them.

@MarkEWaite
MarkEWaite requested a review from a team as a code owner December 31, 2025 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Reduces maintenance effort by changes not directly visible to users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants