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.

Sign up for GitHub

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

Jump to bottom

autoindex in location.pp does not work #304

Closed
andyleejordan opened this issue Apr 24, 2014 · 1 comment
Closed

autoindex in location.pp does not work #304

andyleejordan opened this issue Apr 24, 2014 · 1 comment

Comments

@andyleejordan
Copy link
Contributor

Specifically with version 0.0.9. I've checked and the variable gets defined properly and has the right value, the template (specifically vhost_location_directory.erb) does not insert the autoindex parameter. It is declared in there properly, so my only guess is a syntax error (perhaps a stray <% end -%>) before it. I know that at least the prepend hash works.

I'm trying to track it down, but haven't found it yet.
@andyleejordan
Copy link
Contributor Author

Aight, actually what threw me was all of the templates get used at some point, so I had to use another test to figure out which template it was (false positive on vhost_location_directory.erb). It is truly using vhost_location_directory.erb (as I have a location_alias defined), and it is that template which is missing the autoindex option. Adding it works, but my question is, is this the right thing to do? Should a location alias not ever use the autoindex option, or was it simply forgotten? I'm looking through the Nginx docs right now.

@andyleejordan andyleejordan mentioned this issue Apr 24, 2014
Merged
@jfryman jfryman closed this as completed Apr 24, 2014
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this issue Oct 23, 2017
@andyleejordan
Adding autoindex to location alias
a173a3c 
Fix for issue voxpupuli#304

I don't see any reason why autoindex shouldn't be used with alias
directive, nor does jaybe on #nginx.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jfryman @andyleejordan