Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use core default driver for XStream #1390

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

basil
Copy link
Member

@basil basil commented Sep 21, 2023

The XStream default driver (based on XML Pull Parser 3rd edition) is one that core no longer prefers or uses for various reasons, including bugs parsing emoji in modern XML files. Instead core now uses the Streaming API for XML (StAX) that is built into modern versions of the Java Platform runtime environment. I have been working to slowly remove the old XPP3 parser from core, but I discovered that this plugin still had a hard-coded reference to it in one place and was broken when I removed the old driver from core. To avoid this problem I am instead having this plugin use core's default driver, which in newer versions is StAX.

Testing done

I ran this plugin's tests against a core with XPP3 removed and observed they started failing. After this PR and jenkinsci/nested-view-plugin#43 they now pass.

Submitter checklist

@basil basil added the internal label Sep 21, 2023
@basil basil requested a review from a team as a code owner September 21, 2023 23:44
@jamietanna
Copy link
Contributor

@basil want me to merge? Or you happy to?

@basil
Copy link
Member Author

basil commented Sep 27, 2023

I am an org admin but not a maintainer of this plugin, and org admins generally do not merge changes for plugins they are not maintaining (even though they have the permissions to) as part of our social contract, unless some exception presents itself.

@jamietanna
Copy link
Contributor

All good, thought I'd ask! Would it be handy to have it released sooner than later?

@jamietanna jamietanna merged commit 59f4009 into jenkinsci:master Sep 27, 2023
15 checks passed
@basil basil deleted the default-driver branch September 27, 2023 15:32
@basil
Copy link
Member Author

basil commented Sep 27, 2023

Yes, it would be nice to get this cleaned up and released so that core can eventually remove the outdated driver once all consumers have been eliminated.

@jamietanna
Copy link
Contributor

@basil all released now, thanks for your patience 🚀

@basil
Copy link
Member Author

basil commented Oct 16, 2023

Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants