diff options
author | Eric Wong <e@80x24.org> | 2021-10-14 04:32:54 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-10-14 09:07:16 +0000 |
commit | eb0e4dd09d540eb29c71fe1cc92161a13a0b608a (patch) | |
tree | 1051f61a3db1b62dd99d21e67d6e6be00c31611e /lib/PublicInbox/LEI.pm | |
parent | 896c59c925e61caf5d985e8531a35825eeecc99d (diff) | |
download | public-inbox-eb0e4dd09d540eb29c71fe1cc92161a13a0b608a.tar.gz |
While lei is intended for non-public mail and runs umask(077) by default, externals are one area which can safely defer to the user's umask. Instead of sending it unconditionally with every command, only have lei-daemon request it when necessary.
Diffstat (limited to 'lib/PublicInbox/LEI.pm')
-rw-r--r-- | lib/PublicInbox/LEI.pm | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/lib/PublicInbox/LEI.pm b/lib/PublicInbox/LEI.pm index bd8a6bef..635cd0c5 100644 --- a/lib/PublicInbox/LEI.pm +++ b/lib/PublicInbox/LEI.pm @@ -1518,4 +1518,15 @@ sub cfg_dump ($$) { undef; } +sub request_umask { + my ($lei) = @_; + my $s = $lei->{sock} // return; + send($s, 'umask', MSG_EOR) // die "send: $!"; + vec(my $rvec = '', fileno($s), 1) = 1; + select($rvec, undef, undef, 2) or die 'timeout waiting for umask'; + recv($s, my $v, 5, 0) // die "recv: $!"; + (my $u, $lei->{client_umask}) = unpack('AV', $v); + $u eq 'u' or warn "E: recv $v has no umask"; +} + 1; |