sous-chefs/nginx

Support for pre-v10 attributes

piloos opened this issue ยท 2 comments

๐Ÿ‘ป Brief Description

In release 10, the cookbook switched from being attributes based to being resource based. How do we support configuration options (eg. client_max_body_size) which are not present in the resource? It seems that the only way is to create your own nginx.conf template which seems cumbersome and error prone: suddenly we need to specify all nginx parameters and support the configuration file for newer versions of nginx
Am I missing something in the philosophy of the attributes vs resource switch?

๐Ÿฅž Cookbook version

10.0.2

๐Ÿ‘ฉโ€๐Ÿณ Chef-Infra Version

15.6.10

๐ŸŽฉ Platform details

Ubuntu 16.04

This looks like a feature request, where we just need to add that property to the resource. We should be covering it, if it's a highly used one (max_body_size seems like it should be!)

If you want to get a list together, we can add a bunch to a PR and get this fixed. Better yet, feel free to open a PR with docs and the feature and it'll get shipped!

The latter is more likely to happen as I'm currently rewriting Java, and we could always use more contributions!

Thanks,
Dan

I'm going to close this one down for a more focused feature request. If anyone wants a