Relax or improve the syntax check on proxy_cache_levels

[jrrbelmont]

Currently init.pp checks that the value of proxy_cache_levels "isInteger()". nginx allows proxy_cache "levels" values like "1:2" and "1:2:2". This valid syntax is currently not allowed with this mod.

Suggested fix: Remove the "isInteger()" check for proxy_cache_levels in init.pp and replace it with validate_string(). Better fix would be to parse the string and do a better syntax check.

[recastrodiaz]

+1
Also, listen_port in vhost.pp should allow to set the listen port to something like:

listen_port => '80 default_server',

https://github.com/jfryman/puppet-nginx/blob/master/manifests/resource/vhost.pp#L202

[3flex]

@recastrodiaz you can already set default_server with the listen_options parameter. See https://github.com/jfryman/puppet-nginx/blob/master/templates/vhost/vhost_header.erb#L2 for how it's rendered in the template.

[recastrodiaz]

@3flex thanks for the tip!