Bug 677033 - Default vhost should be in a conf.d file
Summary: Default vhost should be in a conf.d file
Keywords:
Status: CLOSED DUPLICATE of bug 635776
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: nginx
Version: el5
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jeremy Hinegardner
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-02-12 17:48 UTC by Charles Duffy
Modified: 2011-02-13 21:24 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-13 21:24:44 UTC
Type: ---


Attachments (Terms of Use)

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 ***


Note You need to log in before you can comment on or make changes to this bug.