Opened 7 years ago

Closed 7 years ago

#1198 closed enhancement (fixed)

debian postrm script deletes log folder

Reported by: vlajos@… Owned by: thresh
Priority: minor Milestone:
Component: other Version: 1.10.x
Keywords: debian Cc: nginx@…
uname -a: not relevant
nginx -V: not relevant

Description

Hi,

I played a bit with the "official" debian package and realised that when I removed it, it removed the whole /var/log/nginx directory. (The debian packages don't do this.)
I think this is a bit dangerous...
(Otherwise maybe it would worth considering converging the debian scripts between debian and upstream, but I guess this is a bigger task.)

You can double check it here:
​http://nginx.org/packages/debian/pool/nginx/n/nginx/nginx_1.10.3-1~jessie.debian.tar.xz

nginx.postrm:

remove)

rm -rf /var/cache/nginx /var/log/nginx
;;


Thank you!

Best regards,
Lajos Veres

Change History (2)

comment:1 by thresh, 7 years ago

Owner: set to thresh
Status: new → assigned

comment:2 by thresh, 7 years ago

Resolution: → fixed
Status: assigned → closed

Thanks. Fixed for upcoming 1.11.x releases: ​https://hg.nginx.com/pkg-oss/rev/f4717b04fd54

If we choose to release yet another 1.10.x, it will also be there.

Note: See TracTickets for help on using tickets.