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