Bug 677033

Summary: Default vhost should be in a conf.d file
Product: [Fedora] Fedora EPEL Reporter: Charles Duffy <charles>
Component: nginxAssignee: Jeremy Hinegardner <jeremy>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: el5CC: allisson, jeremy
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-02-13 21:24:44 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Charles Duffy 2011-02-12 17:48:01 UTC
Description of problem:

  The default vhost (with the successful-server-installation message) is included in the base nginx.conf file, and cannot be deleted or overridden (except by more specific, non-default definitions) in conf.d/* files.

  Moving the default vhost into a conf.d file would make it easier for users to deploy different default vhosts without needing to modify the base configuration.

Version-Release number of selected component (if applicable):

  nginx-0.8.53-1.el5

How reproducible:

  100%

Steps to Reproduce:
1. Install yum package
2. Add a new default server definition in /etc/nginx/conf.d/
3. Restart server
4. Request page
  
Actual results:

  The page from the default your-server-was-successfully-installed vhost defined in /etc/nginx.conf is displayed

Expected results:

  The page from the dropped-in configuration should be used. (Requiring the user to delete or overwrite a specific file, ie. /etc/nginx/conf.d/default.conf, would be reasonable).

Comment 1 Charles Duffy 2011-02-12 18:48:23 UTC
Reordering the line "include /etc/nginx/conf.d/*.conf;" to be above the default server definition would be an even easier fix.

Comment 2 Jeremy Hinegardner 2011-02-13 21:24:44 UTC
This is a duplicate of #635776 and is already fixed in rawhide.

*** This bug has been marked as a duplicate of bug 635776 ***