#3128 - Redirect Manager import from xlsx not working

[lollegp]

(#3128)

[codecov]

Codecov Report

Merging #3129 (2fab221) into master (3698e61) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 2fab221 differs from pull request most recent head 94846fe. Consider uploading reports for the commit 94846fe to get more accurate results

@@            Coverage Diff            @@
##             master    #3129   +/-   ##
=========================================
  Coverage     55.18%   55.18%           
+ Complexity     5405     5404    -1     
=========================================
  Files           711      711           
  Lines         28988    28988           
  Branches       3745     3745           
=========================================
  Hits          15997    15997           
  Misses        11488    11488           
  Partials       1503     1503           

Impacted Files	Coverage Δ
...mmons/mcp/impl/ProcessErrorReportExcelServlet.java	100.00% <100.00%> (ø)

[YegorKozlov]

Thanks for the PR. This class was impacted by the POI version upgrade, but it's not used by redirect manager.

The classes to fix are com/adobe/acs/commons/redirects/servlets/ExportRedirectMapServlet.java and com.adobe.acs.commons.redirects.servlets.ImportRedirectMapServlet

[lollegp]

Thanks, you are right, I add the fix in the right files and also in another one spotted in the GenericReportServlet.

[YegorKozlov]

LGTM