Comment 6 for bug 1433390

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-swift (stable/icehouse)

Reviewed: https://review.openstack.org/167306
Committed: https://git.openstack.org/cgit/stackforge/puppet-swift/commit/?id=392ab49664f5852bef53ceac5683c0f2a48149cf
Submitter: Jenkins
Branch: stable/icehouse

commit 392ab49664f5852bef53ceac5683c0f2a48149cf
Author: David Moreau Simard <email address hidden>
Date: Tue Mar 17 22:05:53 2015 -0400

    Provide a mean to change the default rsync chmod

    puppet-rsync provides a default chmod of 0644.
    puppet-swift, until this commit, did not provide a way
    to change that default chmod.
    According to the experience in bug #1433390, it
    seemed possible for folders to be created in 0644, thus
    denying access to the folders unless you were root.

    Backwards compatibility is maintained through the same
    defaults.. It looks like the default is broken in
    the context of Swift (unless it runs as root?), however.
    We should consider putting 'Du=rwx,g=rx,o=rx,Fu=rw,g=r,o=r'
    which translates to 755 for folders and 644 for files.

    Change-Id: I7fcef6e4cdf478e808de2895168165989ff098e4
    Closes-bug: #1433390
    (cherry picked from commit 2caba862a8609528f10c975d7a956b246f2ca477)