Redirect Manager - Import from excel not working due to "Apache Poi" dependencies upgrade

[lollegp]

Required Information

• AEM Version: AEM Cloud SDK 2023.6.12255.20230608T053118Z
• ACS AEM Commons Version: 6.0.10
• Reproducible on Latest? yes

Expected Behavior

In redirect manager I should be able to import a set of redirect from an excel file

Actual Behavior

In redirect manager when I try to import a set of redirect from an excel file I'm getting 500 with an exception and can't proceed with the import

Steps to Reproduce

Tools --> ACS AEM Commons --> Manage Redirects --> Choose one conf (not relevant which one) --> Import --> Upload xlsx file (even if you upload an empty one is not working) --> Import Redirect Map --> FAIL

Links

[lollegp]

After a quick analysis it seems that with the new AEM Cloud SDK the "Apache Poi" bundle has been updated and in the new version all deprecated methods have been removed and should be replaced with some new available methods.

[YegorKozlov]

I confirmed the issue with aem-sdk-2023.6.12441.20230623T133322Z and AEM 6.5.17

ACS commons does not control the version of Apache POI - Apache POI is a transitive dependency of Apache Tika which is used internally by Jackrabbit to index binary data.

It appears the POI version has been upgraded from 4.0.1 to 5.2.3 which is quite a leap. And it impacted every place where we read data from spreadsheets: Redirect Manager, MCP jobs, reports, etc .

The minimal version of AEM supported by ACS Commons is 6.5.10, which means we need to support both versions: v4.0.1 and v5.2.3.

[lollegp]

Thanks, Here the PR #3129 to fix the issue for Redirect Manager. It is a simple replacement of the deprecated method with an alternative one. The methods were already deprecated in v 4.0.1 so it should be retro-compatible.

[davidjgonzalez]

Fixed in 6.0.12