Bug 1000616 - NodeJS setting .npm and .npmrc to wrong owner
Summary: NodeJS setting .npm and .npmrc to wrong owner
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Online
Classification: Red Hat
Component: Containers
Version: 2.x
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Hiro Asari
QA Contact: libra bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-23 20:47 UTC by Jhon Honce
Modified: 2015-05-14 23:27 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-08-29 12:55:08 UTC
Target Upstream Version:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Jhon Honce 2013-08-23 20:47:14 UTC 
Description of problem:
The install script using $OPENSHIFT_APP_UUID.$OPENSHIFT_APP_UUID which will only be correct on the head gear.  $OPENSHIFT_GEAR_UUID is the correct value to use.

Additionally, script is calling chown without first sanitizing the UUID's with id.  Should be using chown `id -u $OPENSHIFT_GEAR_UUID`



Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
Comment 1 Hiro Asari 2013-08-26 13:51:41 UTC 
https://github.com/openshift/origin-server/pull/3496
Comment 2 Qiushui Zhang 2013-08-28 05:01:01 UTC 
Tested on devenv_3716.

Create an app of nodejs-0.6. Check the install scripts. It is already using "OPENSHIFT_GEAR_UUID". 

Mark the defect as verified.
Note You need to log in before you can comment on or make changes to this bug.