Uploaded image for project: 'Buildr (Retired)'
  1. Buildr (Retired)
  2. BUILDR-143

Upload to a file:// path needs ability to specify permissions

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.2.10
    • 1.4.2
    • Core features
    • None
    • Linux(RHEL 4) with a samba mount point

    Description

      When uploading an artifact to a file:// repository URL, the permissions on the artifact are rw by owner, but unreadable by anyone else. When I deploy to a samba share, this is a problem, because no one else can read the artifact. I've tracked it down to the use of Tempfile in transports.rb. Tempfile always creates a file with permission 600. If I add the following code, everything works as expected:

      temp.chmod(0644)

      I'd prefer it if the artifact was created using my default umask, and then allow me to override it with an option :permissions so that I can specify what file mode I want. So a better fix would be the following:

      temp.chmod(0666 - File.umask)

      This could be related to issue BUILDR-23

      Attachments

        1. 001_preserve_permissions_spec.patch
          0.6 kB
          Joel Muzzerall
        2. 001_preserve_permissions_code.patch
          0.5 kB
          Joel Muzzerall

        Issue Links

          Activity

            People

              Unassigned Unassigned
              jwr Jared Robinson
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: