[PATCH] Support FreeBSD jails for testing

Maxim Dounin mdounin at mdounin.ru
Tue Oct 20 01:52:10 UTC 2015


Hello!

On Mon, Oct 19, 2015 at 11:39:27PM +0100, Steven Hartland wrote:

[...]

> >>That doesn't fix the directory permission issue which causes pretty much
> >>every test to fail, so is this still an option for inclusion?
> >Directory premissions may vary depending on umask used.  If your
> >umask breaks tests for you - you may try changing it while running
> >tests.  It also shouldn't be important when running tests under
> >non-privileged user.
> Confirmed it doesn't effect a non-privileged user.
> 
> The default umask is 022 hence the issue if its run as privileged user with
> the default user as nobody which will have no access to the created
> directory.

Err, it looks like umask is unrelated here - it's File::Temp which 
creates temp directory with 0700 access mode.

[...]

> Something to mention in the README, if you don't like the chmod?

For sure we don't want chmod(0777), as an world-writable temp 
directory can easily result in problems when running tests on a 
host with untrusted users.  We can consider something like 
chmod(0755), but I'm not sure it's a good idea either, and this 
won't fix at least some tests.

As for the README, it already suggests to run under a 
non-privileged user - it uses "$ " shell prompt in the usage 
example.  ;)  If you think it's not enough and needs to be said 
explicitly - patches are welcome.

-- 
Maxim Dounin
http://nginx.org/



More information about the nginx-devel mailing list