Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 321747 - net-proxy/tinyproxy-1.8.1: permissions and config file path bugs
Summary: net-proxy/tinyproxy-1.8.1: permissions and config file path bugs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Network Proxy Developers (OBSOLETE)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-28 01:06 UTC by Alexander Orlenko
Modified: 2010-06-03 23:07 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to fix all issues (tinyproxy-1.8.1.ebuild.patch,1.44 KB, patch)
2010-05-28 01:16 UTC, Alexander Orlenko
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Orlenko 2010-05-28 01:06:06 UTC
Tinyproxy 1.6 package default config file installed in /etc/tinyproxy/tinyproxy.conf, but in 1.8 package - in /etc/tinyproxy.conf.
Path of config file hardcoded in init script for all branches (/etc/tinyproxy/tinyproxy.conf) thats is an error for 1.8 branch.

Second issue - we need to add user for the tinyproxy daemon and create dirs for pidfile and logfiles and fix permissions.


Reproducible: Always
Comment 1 Alexander Orlenko 2010-05-28 01:16:28 UTC
Created attachment 233193 [details, diff]
Patch to fix all issues

This patch fix next issues:
1) Add tinyproxy user/group and replace nobody user in config
2) Change path of default location of config file/log file/pid file
3) Keep /usr/share/tinyproxy dir, because it contains some helpful html files and this path is defined in config
4) Create dirs for pid file and log files and fix permissions
5) Some improvements and fixes
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2010-06-03 23:07:47 UTC
Hm, should have looked at your patch before I went to work on it myself. I've fixed all of them in 1.8.1-r1 I think, except the sysconfdir change - if /etc is the default now, then we'll use that (I patch the initd script instead). Thanks for reporting! Reopen if I turn out to have left some issues unaddressed.