Skip to content

Options param not available for vhosts #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
luishdez opened this issue Jan 22, 2013 · 2 comments
Open

Options param not available for vhosts #13

luishdez opened this issue Jan 22, 2013 · 2 comments

Comments

@luishdez
Copy link

Not sure if this is possible or I misunderstood the concept of the nexGen Modules, but I'm trying to pass some options usign nginx:vhosts { … } to use in my custom template but this is not possible. I get Invalid parameter options at …

It's not the concept of these modules to be customizable ? How this can be achieved ?
I've seen that I can set some options in the nginx main class but obviously the important thing is to set them in the vhost file. I can do it setting the vars outside but the problem comes when you want to set multiples vhosts.

Sorry if this is possible but I couldn't find it in your web and manuals.

Thanks for sharing ;) these are great.

@alvagante
Copy link
Contributor

Actually the option_lookup function currently works only for the main class of module and not, as in this case, for a subclass or a define.
I do agree, anyway, that it would be interesting to have it also for defines like nginx::vhost (given that is added the 'options' parameter), so I would consider this an open feature request.

@egeland
Copy link
Contributor

egeland commented Jun 25, 2013

+1 for this - my colleague just brought it to my attention that we can't pass the options to the vhost, which would be great if we could..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants