darcs

Issue 2725 _darcs/patch_index: darcs removes all permissions for group and others

Title _darcs/patch_index: darcs removes all permissions for group and others
Priority Status resolved
Milestone Resolved in 2.18.1
Superseder Nosy List bfrk
Assigned To
Topics

Created on 2024-06-01.10:54:22 by bfrk, last changed 2024-06-08.17:10:34 by noreply.

Messages
msg23924 (view) Author: bfrk Date: 2024-06-01.10:54:21
This is due to the use of withPermDir (followed by a rename), which 
creates the temporary directory with these permissions for security 
reasons. We should use our umask to (recursively) set the permissions 
before renaming the temporary directory back to the actual one.
msg23987 (view) Author: noreply Date: 2024-06-08.17:10:34
The following patch sent by Ben Franksen <ben.franksen@online.de> updated issue issue2725 with
status=resolved;resolvedin=2.18.1

Hash: 3979f2a1a11d913aeb8bbecb4c821db4ec75a6c0
Author: Ben Franksen <ben.franksen@online.de>
* resolve issue2725: permissions of _darcs/patch_index

  The cheap solution here is to set the permissions of the newly created
  _darcs/patch_index/ to that of its parent directory _darcs/. Using _darcs/
  has the advantage that darcs now automatically fixes the permissions of
  existing patch_index directories.
History
Date User Action Args
2024-06-01 10:54:22bfrkcreate
2024-06-01 11:11:34bfrksettitle: _darcs/patch-index: darcs removes all permissions for group and others -> _darcs/patch_index: darcs removes all permissions for group and others
2024-06-08 17:10:34noreplysetstatus: unknown -> resolved
messages: + msg23987
resolvedin: 2.18.1