Add autoindex to ssl_header too

[bc-bjoern]

Hello,

the option autoindex was missing in server_ssl_header.erb.

Thanks!

[ekohl]

Could you add a test for this?

[bc-bjoern]

May be...

[bc-bjoern]

Okay, I can't.

[ekohl]

Feel free to keep the PR open and add commits to it. If you need help with writing a test we're happy to help.

[bc-bjoern]

I do not understand this unfortunately. I guess I need to add it in spec/defines/resource_server_spec.rb in line 439 "describe server_ssl_header template content". But I do not understand how to do it correctly.

[ekohl]

It would be very similar to:

puppet-nginx/spec/defines/resource_server_spec.rb

Lines 836 to 841 in ae2af80

	{
	title: 'should set index_file(s)',
	attr: 'index_files',
	value: %w[name1 name2],
	match: %r{\s*index\s+name1\s+name2;}
	},

That's where we test the SSL part of it. You can copy that one and modify it to autoindex instead of index_files.

[bc-bjoern]

Sure, this part is clear more or less.

           [    
             {    
               title: 'should set autoindex',
               attr: 'autoindex',
               value: 'on',
               match: '  autoindex on;'
             }    
           ] 

But I didn't managed it to include this into the server_ssl_header section.

[bc-bjoern]

Travis says no

[ekohl]

Thanks!