Skip to content

Commit

Permalink
Revert the modification of permissions on the upload directory.
Browse files Browse the repository at this point in the history
Correct default permissions are more easily achieved via file ACLs when
the gobbler maintainer sets up the staging directory.
  • Loading branch information
LTLA committed Sep 18, 2024
1 parent 99916a2 commit f145d7f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
4 changes: 2 additions & 2 deletions DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: gobbler
Version: 0.3.7
Date: 2024-09-15
Version: 0.3.8
Date: 2024-09-18
Title: Interface to the gobbler service
Description:
Friendly interface to the gobbler service.
Expand Down
6 changes: 0 additions & 6 deletions R/uploadDirectory.R
Original file line number Diff line number Diff line change
Expand Up @@ -80,12 +80,6 @@ uploadDirectory <- function(project, asset, version, directory, staging, url, pr

dump_request(staging, url, "upload", req)

# Once the upload is done, we make the directory world-writeable so the
# backend can more easily purge old files. We could also delete them right
# away but sometimes it's helpful for debugging to leave them there.
subdirs <- list.dirs(directory, full.names=TRUE, recursive=TRUE)
Sys.chmod(subdirs, mode="0777", use_umask=FALSE)

invisible(NULL)
}

Expand Down

0 comments on commit f145d7f

Please sign in to comment.