[OFBIZ-12936] ‎SecuredUpload‎::checkMaxLinesLength put in a regression for PDF files

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 18.12.13
• Fix Version/s: 18.12.13
• Component/s: framework/security
• Assignee: Jacques Le Roux
• Reporter: Jacques Le Roux
• Created: 12/Mar/24 06:03
• Updated: 14/Mar/24 08:07
• Resolved: 14/Mar/24 08:07

Description

As commented at https://s.apache.org/wpkuj, while working on OFBIZ-12926 I realised that checkMaxLinesLength put in a possible regression for PDF files.

As a summary: we can't use checkMaxLinesLength for PDF files because they may contain several CharSet encodings. Hence no possibility to use Files::readAllLines that needs a sole CharSet. So we need to bypass this check.