Bug 484475 - Change package permissions to rw-r--r--
Summary: Change package permissions to rw-r--r--
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Satellite 5
Classification: Red Hat
Component: Satellite Synchronization
Version: 422
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Pradeep Kilambi
QA Contact: Jan Hutař
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-02-07 00:06 UTC by Xixi
Modified: 2018-10-19 23:50 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of: 456562
Environment:
Last Closed: 2009-02-24 17:07:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Xixi 2009-02-07 00:06:13 UTC
Cloning for Satellite as there has been a related customer issue reported.

+++ This bug was initially created as a clone of Bug #456562 +++

When porting the package download from perl to java, tomcat will need to be able
to read all of the pacakges, but sat-sync and rhnpush currently marks the
packages as 640 with apache:root ownership.

This needs to be changed to 644.

--- Additional comment from cperry on 2008-08-01 14:46:21 EDT ---

Any other viable options? 
- Make tomcat user in the apache group *and* then change sat-sync code to write
stuff:
0640 apache:apache
from
0640 apache:root

Does 'root' need this access at group level to the files? 

Making the files globally readable by change:
0644 
is less of a change to do, less risky in code, but does expose the entire
/var/satellite/ contents to any normal user on that has ssh access, but is this
a problem, most likely not. 

So, 0644 sounds fine, I think the risk exposure is minimal. 

Pretty sure this is a one liner to change:
satellite_tools/syncLib.py:        setPermsPath(self.full_path, user='apache',
group='root', chmod=0640)

*if* we make a change, we will need as part of upgrade process, a separate bug
to track and if needed as part of upgrade change permissions. 


Cliff

--- Additional comment from pkilambi on 2008-08-01 15:12:37 EDT ---

fixed and upgrade process already handles this perms update.

--- Additional comment from dgoodwin on 2008-09-05 11:44:28 EDT ---

Verified in spacewalk 0.2. Packages are being stored with 0644. (-rw-r--r--)

Comment 5 James M. Leddy 2009-02-12 20:02:38 UTC
repsonding to comment #3

Prad,  They're basically asking for apache/apache 664 because they have scripts that run as non-apache user and they want to write as apache group.  Is that possible?

Comment 7 James M. Leddy 2009-02-12 21:21:27 UTC
GS would like 664, as in group read/write

Comment 16 Issue Tracker 2009-03-12 18:54:56 UTC
Closing this as per today's meeting.  Goldman would have liked to see this
changed.

Internal Status set to 'Resolved'
Status set to: Closed by Tech
Resolution set to: 'Rejected'

This event sent from IssueTracker by jleddy 
 issue 218040


Note You need to log in before you can comment on or make changes to this bug.