Skip to content
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

Add nginx::vhost option 'autoindex' #194

Merged
merged 1 commit into from
Dec 12, 2013
Merged

Add nginx::vhost option 'autoindex' #194

merged 1 commit into from
Dec 12, 2013

Conversation

bionix
Copy link
Contributor

@bionix bionix commented Dec 12, 2013

I implemented the option to set in the nginx::vhost the option autoindex = on.

Details about the function "autoindex": http://wiki.nginx.org/HttpAutoindexModule

Signed-off-by: Markus Rekkenbeil [email protected]

Signed-off-by: Markus Rekkenbeil <[email protected]>
jfryman pushed a commit that referenced this pull request Dec 12, 2013
Add nginx::vhost option 'autoindex'
@jfryman jfryman merged commit 1233413 into voxpupuli:master Dec 12, 2013
@jfryman
Copy link
Contributor

jfryman commented Dec 12, 2013

Nice. Thanks for the code! 🙇

@bionix bionix deleted the bionix_autoindex branch December 13, 2013 14:02
@bionix bionix restored the bionix_autoindex branch December 16, 2013 16:15
@bionix bionix deleted the bionix_autoindex branch December 16, 2013 16:49
@sc68cal
Copy link

sc68cal commented Feb 7, 2014

I'm having trouble getting this feature to work - here's the relevant section of my Puppet config. www_root is getting set correctly, while autoindex is not. It could be that I'm a puppet newbie and have made a simple mistake, but would appreciate some help.

nginx::resource::vhost { 'foo.net':
  ensure => present,
  www_root => $log_http_root,
  listen_port => 80,
  autoindex => 'on',
}

@jfryman
Copy link
Contributor

jfryman commented Feb 10, 2014

@bionix Any input here? ^^^

@bionix
Copy link
Contributor Author

bionix commented Feb 10, 2014

I'm checking it later.

@bionix
Copy link
Contributor Author

bionix commented Feb 10, 2014

I found the problem.
Please add my pull request to fix this issue.
#255

@jfryman
Copy link
Contributor

jfryman commented Feb 10, 2014

@sc68cal @bionix was kind enough to provide a bug fix that I've merged in. Would you see if you're 👍 now?

@sc68cal
Copy link

sc68cal commented Feb 10, 2014

I can confirm the fix - thanks a lot!

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

Successfully merging this pull request may close these issues.

3 participants