[NUTCH-2509] Inconsistent behavior in SitemapProcessor

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.14
• Fix Version/s: 1.15
• Component/s: sitemap
• Patch Info: Patch Available
• Assignee: Unassigned
• Reporter: Yossi Tamari
• Created: 13/Feb/18 16:51
• Updated: 01/Oct/19 14:29
• Resolved: 02/Apr/18 16:09

Description

There are two inconsistent behaviors in SitemapProcessor:

1. There is a member variable maxRedir that is supposed to limit the number of redirections on sitemap URLs, and it is initialized from config property sitemap.redir.max, but it is ignored in the code because a local variable with the same name is defined in the relevant method, and is always set to 3.
2. When a sitemap URL goes through redirect, it is filtered and normalized. However, if a sitemap URL comes from a sitemapindex, it is not. This seems inconsistent, as in both cases we have a URL from an outside source.

Issue Links

• links to: GitHub Pull Request #301