<<< Date Index >>>     <<< Thread Index >>>

Re: [PATCH] Add $umask for mailboxes and attachments



Re: Thomas Roessler 2007-03-20 
<20070320093535.GG30802@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
> >$umask defaults to 077. It's up to the user to override it. But if
> >the user wants to, it's more convenient to do it in mutt than to
> >suspend or quit and navigate to the created folder (and its
> >subdirectories if it is maildir) to fix up the permissions
> >afterward, IMHO.

> Yes.

Ack.

> But you have to weigh the downsides of the change against this:

> - Code messiness.  (And yes, this is a significant consideration in
>   this.)

> - Introducing a configuration variable for an operation that doesn't
>   call for changing the default, but (possibly) for a case-by-case
>   change.
>   I think that the code messiness alone (together with the old code
> taking care of most relevant use cases) would warrant not taking
> this patch in.

Nonetheless, the patch is in now, and we have a lengthy thread about
it. How do we proceed from here?

I would like to retain the ability to configure the umask (of course
it should default to 077) and to resolve the code messiness. I think
my patch does that, modulo the question if all temp files are properly
created with 0600 permissions (IMHO the are). Another question is
/var/mail - does mutt ever create mboxes there or is that always the
MTA's job?

If the latter 2 questions cannot be resolved, it is probably best to
remove the umask code again. Though IMHO it should be possible to
solve them.

Christoph
-- 
cb@xxxxxxxx | http://www.df7cb.de/

Attachment: signature.asc
Description: Digital signature