reopen 610290
thanks

DBTS> I'm closing this bug because it is invalid.

DBTS> I've never heard of anyone using port 80 for an internal listener. Most 
other

The bugreport isn't about an using listener. The problem: upgrade
process restores /etc/nginx/sites-available/default file if this file
was deleted by user. User can have many causes to remove this file.
I showed one *EXAMPLE* of deleting this file.
Don't close the bug because it isn't 'invalid'.
If You don't like this example I can show another.

You can reproduce the bag easy:

apt-get install nginx/squeeze
rename or delete /etc/nginx/sites-available/default
upgrade nginx upto sid

see here:
vds.uvw.ru:[/home/dimka]# apt-cache policy nginx
nginx:
  Installed: (none)
  Candidate: 0.8.54-3
  Version table:
     0.8.54-3 0
        500 http://debian.uvw.ru sid/main Packages
     0.7.67-3 0
        500 http://debian.uvw.ru squeeze/main Packages

vds.uvw.ru:[/home/dimka]# apt-get install nginx=0.7.67-3
Reading package lists... Done
Building dependency tree       
Reading state information... Done
The following packages were automatically installed and are no longer required:
  libaprutil1-dbd-sqlite3 openssl-blacklist libxslt1.1 libaprutil1-ldap
  libgd2-noxpm ssl-cert libjpeg62 libfreetype6
Use 'apt-get autoremove' to remove them.
The following NEW packages will be installed:
  nginx
0 upgraded, 1 newly installed, 0 to remove and 244 not upgraded.
Need to get 338kB of archives.
After this operation, 799kB of additional disk space will be used.
Get:1 http://debian.uvw.ru squeeze/main nginx 0.7.67-3 [338kB]
Fetched 338kB in 0s (345kB/s) 
Selecting previously deselected package nginx.
(Reading database ... 21620 files and directories currently installed.)
Unpacking nginx (from .../nginx_0.7.67-3_i386.deb) ...
Processing triggers for man-db ...
Setting up nginx (0.7.67-3) ...

vds.uvw.ru:[/home/dimka]# invoke-rc.d nginx restart                  
Restarting nginx: nginx.

vds.uvw.ru:[/home/dimka]# mv /etc/nginx/sites-enabled/default 
/etc/nginx/sites-enabled/default-renamed
`/etc/nginx/sites-enabled/default' -> `/etc/nginx/sites-enabled/default-renamed'

vds.uvw.ru:[/home/dimka]# invoke-rc.d nginx restart                  
Restarting nginx: nginx.

vds.uvw.ru:[/home/dimka]# ls /etc/nginx/sites-enabled -l
total 0
lrwxrwxrwx 1 root root 34 Jan 19 12:09 default-renamed -> 
/etc/nginx/sites-available/default

vds.uvw.ru:[/home/dimka]# invoke-rc.d nginx restart                  
Restarting nginx: nginx.

vds.uvw.ru:[/home/dimka]# apt-get install nginx=0.8.54-3
Reading package lists... Done                           
Building dependency tree       
Reading state information... Done
The following packages were automatically installed and are no longer required:
  libaprutil1-dbd-sqlite3 openssl-blacklist libaprutil1-ldap ssl-cert
Use 'apt-get autoremove' to remove them.
The following extra packages will be installed:
  nginx nginx-full
Suggested packages:
  ufw nginx-doc
The following NEW packages will be installed:
  nginx-full
The following packages will be upgraded:
  nginx
1 upgraded, 1 newly installed, 0 to remove and 244 not upgraded.
Need to get 0B/415kB of archives.
After this operation, 160kB of additional disk space will be used.
Do you want to continue [Y/n]? 
(Reading database ... 21644 files and directories currently installed.)
Preparing to replace nginx 0.7.67-3 (using .../nginx_0.8.54-3_all.deb) ...
Unpacking replacement nginx ...
dpkg: warning - unable to delete old directory `/etc/nginx/sites-enabled': 
Directory not empty
dpkg: warning - unable to delete old directory `/etc/nginx/sites-available': 
Directory not empty
dpkg: warning - unable to delete old directory `/etc/nginx': Directory not empty
Selecting previously deselected package nginx-full.
Unpacking nginx-full (from .../nginx-full_0.8.54-3_i386.deb) ...
Processing triggers for man-db ...
Setting up nginx-full (0.8.54-3) ...
Installing new version of config file /etc/nginx/nginx.conf ...
Installing new version of config file /etc/nginx/sites-available/default ...
Installing new version of config file /etc/nginx/koi-utf ...
Installing new version of config file /etc/nginx/fastcgi_params ...
Installing new version of config file /etc/nginx/win-utf ...
Installing new version of config file /etc/nginx/mime.types ...
Installing new version of config file /etc/nginx/koi-win ...
Installing new version of config file /etc/init.d/nginx ...
Setting up nginx (0.8.54-3) ...

vds.uvw.ru:[/home/dimka]# ls /etc/nginx/sites-enabled -l
total 0
lrwxrwxrwx 1 root root 34 Jan 19 12:10 default -> 
/etc/nginx/sites-available/default
lrwxrwxrwx 1 root root 34 Jan 19 12:09 default-renamed -> 
/etc/nginx/sites-available/default

vds.uvw.ru:[/home/dimka]# invoke-rc.d nginx restart
Restarting nginx: [warn]: conflicting server name "localhost" on 0.0.0.0:80, 
ignored
nginx.

So If a user renamed or deleted this symlink and then upgraded his
distro upto sid he could receive his server broken.

-- 
... mpd is off

. ''`.                               Dmitry E. Oboukhov
: :’  :   email: un...@debian.org jabber://un...@uvw.ru
`. `~’              GPGKey: 1024D / F8E26537 2006-11-21
  `- 1B23 D4F8 8EC0 D902 0555  E438 AB8C 00CF F8E2 6537

Attachment: signature.asc
Description: Digital signature

Reply via email to