map with hostnames does not drop terminating dot

[www.google.com/accounts/o8/id?id=AItOawnQxODTko53PmY0va90aNN76a_jMUVqVco]

In ngx_http_map_variable function of ngx_http_map_module.c there are lines:
124 len = val.len;
125
126 if (len && map->hostnames && val.data[len - 1] == '.') {
127 len--;
128 }
129
130 value = ngx_http_map_find(r, &map->map, &val);

But 'len' is not used after that, so terminating dot cannot be ignored for map with hostnames as it should.

[Maxim Dounin]

Thank you for report. Actually, the code is dead since introduction in 0.3.16, due to map->hostnames being always set to 0.

[Ruslan Ermilov]

In [4713/nginx]:

map: strip final dot before looking up in a map of hostnames.
(closes #182)

[Maxim Dounin]

In 4786/nginx:

Merge of r4713: map hostnames fix (ticket #182).

The final dot wasn't stripped before looking up in a map of hostnames.