Piotr Sikora PiotrSikora

Organizations

@FRiCKLE
@PiotrSikora

The issue is with the $proxy_host variable, which is empty at the time cache purge runs. Do you have any real reason for using it instead of the $host

@PiotrSikora

I'm not sure whether there is value in keeping this around, but if you want to keep it, then I'd prefer if this was before the check, not after.

@PiotrSikora

Style: extra empty line.

@PiotrSikora

Style: two spaces before max=, please.

@PiotrSikora

Style: two spaces before max=, please.

@PiotrSikora

Style: extra space, not aligned.

@PiotrSikora

I don't think _IPV6 in the name is needed here, it's still a host variable and whether it's IPv6 or not, depends on the value.

@PiotrSikora

Could you change this to: ngx_conf_log_error(NGX_LOG_EMERG, cf, 0, "postgres: IPv6 literals require nginx-1.3.1+"); otherwise it's hard for users…

PiotrSikora commented on issue cloudflare/sslconfig#9
@PiotrSikora

RC4 has been disabled since d6529d3.

PiotrSikora commented on issue cloudflare/sslconfig#4
@PiotrSikora

Stay tuned ;)