From 6781be2141b1730aa7fde0e6fed31825a8d55f50 Mon Sep 17 00:00:00 2001 From: Joey Hess Date: Wed, 2 Sep 2020 12:25:27 -0400 Subject: [PATCH] bug report --- ...directory_special_remote_export_file_mode.mdwn | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 doc/bugs/directory_special_remote_export_file_mode.mdwn diff --git a/doc/bugs/directory_special_remote_export_file_mode.mdwn b/doc/bugs/directory_special_remote_export_file_mode.mdwn new file mode 100644 index 0000000000..0773b9f307 --- /dev/null +++ b/doc/bugs/directory_special_remote_export_file_mode.mdwn @@ -0,0 +1,15 @@ +When exporting a tree to a directory special remote, files are written mode +600. This prevents eg, publishing them by http, and then accessing them +with httpalso special remote. + +`viaTmp` creates a temp file, with temp file perms. Either it should use +umask perms, or all callers that don't explicitly set perms should. + +This also affects some other things, eg hook files written by git-annex +init, and some stuff in ~/.config/git-annex like autostart. + +`withTmpFileIn` also uses openTempFile, and probably its callers do need to +adjust perms if desired since it could be used with a real temp directory. + +There are also a couple of other uses of openTempFile, which need to be +audited for this problem. --[[Joey]]