internal error: Huge input lookup

[tvaughan73]

When reading a large excel file I am getting the warnings/error below. Any idea what would cause this?

Severity: Warning --> XMLReader::expand(): zip:///home/dev/tmp/usertemp645567ee73f18/2023.04_US_Tax_Summary_Extract_05.02.2023.xlsx#xl/worksheets/sheet1.xml:2: parser error : internal error: Huge input lookup /vendor/openspout/openspout/src/Reader/XLSX/RowIterator.php 322
Severity: Warning --> XMLReader::expand(): 0494</v></c><c r="G23314" t="s"><v>7313</v></c><c r="H23314" t="s"><v>56</v></c> /vendor/openspout/openspout/src/Reader/XLSX/RowIterator.php 322
Severity: Warning --> XMLReader::expand():                                                                                ^ /vendor/openspout/openspout/src/Reader/XLSX/RowIterator.php 322
Severity: Warning --> XMLReader::expand(): An Error Occurred while expanding /vendor/openspout/openspout/src/Reader/XLSX/RowIterator.php 322
Severity: error --> Exception: OpenSpout\Reader\XLSX\RowIterator::getCell(): Argument #1 ($node) must be of type DOMElement, bool given, called in /vendor/openspout/openspout/src/Reader/XLSX/RowIterator.php on line 323 /home/dev/public_html/application/vendor/openspout/openspout/src/Reader/XLSX/RowIterator.php 404

[Slamdunk]

Any idea what would cause this?

Nope

[Tikvout]

Hi,
I recently got this same error this morning and managed to fix it.
I'm not 100% what the cause is but it seems to be related to memory caused by input buffering.

My fix was to update the openFileInZip method.
In "src/Reader/Wrapper/XMLReader.php"

Changed:

$wasOpenSuccessful = $this->open($realPathURI, null, LIBXML_NONET);

To:

$wasOpenSuccessful = $this->open($realPathURI, null, LIBXML_COMPACT | LIBXML_PARSEHUGE);

By adding the 2 flags LIBXML_COMPACT and LIBXML_PARSEHUGE.
See PHP documentation here for a better understanding on what each flag does.
https://www.php.net/manual/en/libxml.constants.php

I hope this helps.

[tvaughan73]

@Tikvout Thank You!!! this worked. @Slamdunk would this be a change you would consider? As a side note, making this change increased the speed dramatically.

[Slamdunk]

Would you like to propose a PR with such change?

My only note is to keep LIBXML_NONET flag please.

[tvaughan73]

Yes, just put in a PR for the change. Thanks!

[Slamdunk]

@Tikvout have a look at #150 (comment)