[Patch] Make resolver port configurable

Maxim Dounin mdounin at mdounin.ru
Tue Feb 15 00:15:18 MSK 2011


On Mon, Feb 14, 2011 at 03:53:47PM -0500, Sirsiwal, Umesh wrote:

> Thanks for the feedback. Here is update patch. This does not 
> need a second parameter. Just allows host:port based 
> configuration.

Just my 2 cents:

1. You may want to follow style to make all of us happy.

2. Using ngx_parse_url() instead of reinventing the wheel is a 
good idea.

Maxim Dounin

More information about the nginx-devel mailing list