Feature #9379: Support for using libxml-ruby as XML parser in xmlrpc-libs

• Status: Closed
• Assignee: hsbt (Hiroshi SHIBATA)
• Target version: -

[herwinw (Herwin Quarantainenet)]

The default backend in the XMLRPC parser is REXML. This should work at most occasions, but it definitely isn't the fastest implementation. The attached patch allows a backend using libxml-ruby http://xml4r.github.io/libxml-ruby/ which should be more efficient. It successfully passes the unit tests available in the file test/xmlrpc/test_parser.rb

The default isn't changed in this patch, so when libxml-ruby isn't available nothing is changed. Even if it's available, you still have to change the parsing backend before it is used.

[trans (Thomas Sawyer)]

This seems like a very reasonable idea. (Maybe it would even be possible to add support for Nokogiri too?) One thing though, is there a way to not load REXML at all if it is not needed?

[herwinw (Herwin Quarantainenet)]

I guess support for Nokogiri will be possible as well, I just needed something more efficient than REXML and libXML was the first thing I tried, and it just worked.

Regarding your second point: that really needs some restructuring of loading the backends, I think that is a little bit outside the scope of this ticket.

[hsbt (Hiroshi SHIBATA)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

Applied in changeset r47902.

---

• lib/xmlrpc/parser.rb: added new parser class using libxml-ruby gem.
  [Feature #9379][ruby-core:59633]
• lib/xmlrpc/config.rb: ditto.

[zzak (zzak _)]

• Status changed from Closed to Assigned
• Assignee set to hsbt (Hiroshi SHIBATA)

Needs tests

[hsbt (Hiroshi SHIBATA)]

• Status changed from Assigned to Closed

all StreamParser have no tests.