monitorjbl / excel-streaming-reader

An easy-to-use implementation of a streaming Excel reader using Apache POI
Apache License 2.0
960 stars 345 forks source link

new XEE safe XML parser code will fail if parser is not Xerces-based #179

Closed pjfanning closed 5 years ago

pjfanning commented 6 years ago

Some of the parser features are Xerces specific. The Sun JDK parser is a fork of Xerces so that should be ok.

Apache POI dev list regularly gets emails from users who use different Java runtimes (eg IBM) and run into issues due to the XML parser settings.

I made these changes in my fork:

https://github.com/pjfanning/excel-streaming-reader/commit/5e2b37e8d9f6cc90d9d5c0f8ea78a71d4c5c2647

pjfanning commented 5 years ago

Not needed after https://github.com/monitorjbl/excel-streaming-reader/pull/189 was merged