[TS-2269] regex_remap plugin has problem handling the case when url path is empty

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.1.2
• Component/s: Plugins
• Assignee: Leif Hedstrom
• Reporter: Shu Kit Francis Chan
• Created: 10/Oct/13 06:54
• Updated: 05/Dec/13 23:51
• Resolved: 12/Oct/13 15:17

Description

Specifically the block of code is here - https://github.com/apache/trafficserver/blob/master/plugins/regex_remap/regex_remap.cc#L802-806

*(match_buf + match_len) = '/';
if (req_url.path && req_url.path_len > 0)

{ memcpy(match_buf + match_len + 1, req_url.path, req_url.path_len); match_len += (req_url.path_len + 1); }

So if req_url.path is empty (e.g. in the case of http://www.xyx.com/ being the request url), match_len will not increment by 1.

so e.g. there won't be a match for this case for the regular expression of '^/$'