Roundup Tracker - Issues

Issue 1424518

classification
umask (0) too loose
Type: Severity: normal
Components: Installation Versions:
process
Status: closed accepted
:
: richard : nerijus, richard, tonimueller
Priority: normal :

Created on 2006-02-05 13:14 by tonimueller, last changed 2006-02-06 11:03 by nerijus.

Messages
msg2130 Author: [hidden] (tonimueller) Date: 2006-02-05 13:14
Hello,

running with umask(0) imho doesn't help roundup keeping
to its data. Suggested change:

In line 660 in version 1.0.1, file
roundup/scripts/roundup_server.py, change

   os.umask(0) to
   os.umask(077)

Or have a config item in config.ini that's preset to
something less risky.

Thank you!
msg2131 Author: [hidden] (nerijus) Date: 2006-02-06 11:03
Logged In: YES 
user_id=3416

There is a patch (800719) for this issue. 
msg2132 Author: [hidden] (tonimueller) Date: 2006-02-06 14:48
Logged In: YES 
user_id=123575

Hello, I didn't see this patch, but the other one in the
Debian package, and thought it would be a good idea to get
it included with upstream - therefore this bug report.
msg2133 Author: [hidden] (tonimueller) Date: 2006-02-06 14:54
Logged In: YES 
user_id=123575

Ugh. As it turns out, that patch was submitted by my
predecessor who apparently also thought that it would be
nice to get something like this included upstream.
History
Date User Action Args
2006-02-05 13:14:28tonimuellercreate