Missing .conf extension for sites-available/* sites-enabled/*

[hrak]

In this bit of code, the cookbook wants to delete the default vhost config installed by the apache2 package

%w(default 000-default).each do |site|
  link "#{node['apache']['dir']}/sites-enabled/#{site}" do
    action :delete
  end

  file "#{node['apache']['dir']}/sites-available/#{site}" do
    action :delete
    backup false
  end
end

On Ubuntu 14.04 and Debian Sid, the files in sites-available/ and the links in sites-enabled/ have a .conf extension ('000-default.conf') so they are never removed.

[svanzoest]

Thanks @hrak
There are a few different platforms, some with .conf, some without and I can easily see how we may have missed this.

[hrak]

#188 seems related, and also mentions 000-default.conf, but the fix applied in that bug report still does not cover the .conf extension. The only thing that sort of fixed #188 was this change:

+apache_site '000-default' do
+ enable node['apache']['default_site_enabled']
+end

that re-enables the never removed 000-default.conf (ignoring the 'sites-available/default.conf' that gets installed by the cookbook). This bit is now removed from the current version of the cookbook.

[svanzoest]

I think my recent commit resolves this, can you confirm?

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.