Skip to content

Conversation

joakime
Copy link
Contributor

@joakime joakime commented Jul 5, 2023

  • Allow an override of the newSAXParserFactory() to customize the factory.
  • Add new getter getParser() to fetch the active SAXParser.

@joakime joakime added this to the 10.0.x milestone Jul 5, 2023
@joakime joakime self-assigned this Jul 5, 2023
@joakime joakime marked this pull request as draft July 5, 2023 17:04
@joakime joakime marked this pull request as ready for review July 5, 2023 17:05
@joakime joakime merged commit 9a05c75 into jetty-10.0.x Jul 6, 2023
@joakime joakime deleted the fix/10.0.x/xmlparser-getparser branch July 6, 2023 21:32
joakime added a commit that referenced this pull request Aug 11, 2023
…er` in `XmlParser` (#10067)

* Allow customization of SAXParserFactory / SAXParser in XmlParser
* Introduce method `.getSAXParser()`
---------

Signed-off-by: Joakim Erdfelt <[email protected]>
Co-authored-by: Greg Wilkins <[email protected]>
joakime added a commit that referenced this pull request Aug 14, 2023
…er` in `XmlParser` (#10299)

Backports of
* Issue #10066 - Allow customization of `SAXParserFactory` and `SAXParser` in `XmlParser` (#10067)
* Updating various old/moved URL references found across project (`jetty-10.0.x`) (#10098)

Consisting of
* Allow customization of SAXParserFactory / SAXParser in XmlParser
* Introduce method `.getSAXParser()`
* Prepending doctype in testcases
* More comprehensive changes to redirectEntity config
* Updating various old/moved URL references found across project (`jetty-10.0.x`) (#10098)
* Now that the migration of `https://eclipse.org/jetty/` to `https://eclipse.dev/jetty/` has occurred, it is time to review the URI use in our project
* Added more URIs to XmlConfiguration
* Better SAXParseException handling to report resource that is causing the problem
* Add missing DOCTYPE declarations
* Enforcing unique XML ids

---------

Signed-off-by: Joakim Erdfelt <[email protected]>
Co-authored-by: Greg Wilkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow SAXParserFactory or SAXParser to be configured in Jetty's XmlParser class - GHSA-58qw-p7qm-5rvh
3 participants