Bug 699906 - ec2 credentials printed to log file
Summary: ec2 credentials printed to log file
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: imagefactory
Version: 0.3.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
Assignee: Ian McLeod
QA Contact: Aziza Karol
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-04-26 21:51 UTC by Dave Johnson
Modified: 2012-01-26 12:18 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-12-08 13:46:32 UTC


Attachments (Terms of Use)

Description Dave Johnson 2011-04-26 21:51:06 UTC
Noticed in /var/log/imagefactory.log that my ec2 credentials are printed out and visible.  In the past on the conductor frontend logs, we have prevented printing the actual values of the access id and the secret access id fields, we should probably do that here as well. 

See https://bugzilla.redhat.com/show_bug.cgi?id=699883#c3, I didn't notice and accidentally posted my credentials.

Comment 1 wes hayutin 2011-06-23 13:36:24 UTC
need status on this

Comment 2 Ian McLeod 2011-07-06 17:47:22 UTC
I have removed all printing of credentials in the log messages within the Fedora builder (which is also used for RHEL builds).

I have replaced them with generic references to "access_key" or "secret_key".

This is pushed and available as the 0.2.3 interim release here:

http://repos.fedorapeople.org/repos/aeolus/image-factory/0.2.3/

Comment 3 Shveta 2011-07-07 07:26:04 UTC
i see the ec2 credentials still printing to imagefactory.log while building image
Except for secret key all other things are printed :
Accesskey , account number , private key, cert key .

Comment 4 Steve Loranz 2011-07-07 16:55:25 UTC
I've changed the debug statement for QMF method calls to redact the credentials.  You'll see something like the following now:

 args = {'credentials': '*** REDACTED ***', 'image': '99b93b28-f50a-442d-9845-a02044bc23b5', 'build': '6e7db8a4-0932-4b54-98eb-db98b9f2506f', 'providers': ['mock-provider1']} 

I'm prepared to push this out as 0.2.4 unless there are other items from comment 3 above that need action.

Comment 5 Steve Loranz 2011-07-07 17:54:33 UTC
(In reply to comment #4)
> I've changed the debug statement for QMF method calls to redact the
> credentials.  You'll see something like the following now:
> 
>  args = {'credentials': '*** REDACTED ***', 'image':
> '99b93b28-f50a-442d-9845-a02044bc23b5', 'build':
> '6e7db8a4-0932-4b54-98eb-db98b9f2506f', 'providers': ['mock-provider1']} 
> 
> I'm prepared to push this out as 0.2.4 unless there are other items from
> comment 3 above that need action.

This will actually go out in 0.3.0 this weekend.

Comment 6 wes hayutin 2011-07-08 21:19:21 UTC
rpm -q imagefactory before testing

Comment 7 wes hayutin 2011-07-12 18:12:59 UTC
removing from tracker

Comment 8 Aziza Karol 2011-07-13 06:50:55 UTC
[root@dell-pe1950-01 ~]# rpm -qa |  grep imagefactory
imagefactory-0.2.3-1.el6.noarch

ec2 credentials still printing to imagefactory.log while pushing image.

[root@dell-pe1950-01 ~]# rpm -qa  | grep aeolus
aeolus-conductor-0.3.0-0.el6.20110712223242git096643e.noarch
rubygem-aeolus-cli-0.0.1-1.el6.20110711131044git5bc7abf.noarch
aeolus-conductor-daemons-0.3.0-0.el6.20110712223242git096643e.noarch
aeolus-configure-2.0.1-0.el6.20110712153243gite2c11da.noarch
aeolus-all-0.3.0-0.el6.20110712223242git096643e.noarch
aeolus-conductor-doc-0.3.0-0.el6.20110712223242git096643e.noarch 


From comment 5,0.2.4 is yet to come.

Comment 9 wes hayutin 2011-07-14 18:31:27 UTC
2011-07-14 14:30:43,443 DEBUG imagefactory.builders.BaseBuilder.FedoraBuilder pid(18031) Message: Executing register command: euca-register -U http://ec2.us-west-1.amazonaws.com/ -A "access_key" -S "secret_key" imagefactory-ec2-us-west-1-6735-0069-5950/23beb33b-fe68-443c-9f68-eb47b16cf313.manifest.xml

fixed in

[root@hp-z200-06 ~]# rpm -qa | grep aeolus
aeolus-configure-2.0.1-0.el6.20110712153243gite2c11da.noarch
aeolus-conductor-doc-0.3.0-0.el6.20110712223242git096643e.noarch
aeolus-conductor-daemons-0.3.0-0.el6.20110712223242git096643e.noarch
aeolus-all-0.3.0-0.el6.20110712223242git096643e.noarch
rubygem-aeolus-cli-0.0.1-1.el6.20110712223242git096643e.noarch
aeolus-conductor-0.3.0-0.el6.20110712223242git096643e.noarch
[root@hp-z200-06 ~]#

Comment 10 wes hayutin 2011-08-01 19:53:03 UTC
release pending...

Comment 11 wes hayutin 2011-08-01 19:55:26 UTC
release pending...

Comment 13 wes hayutin 2011-12-08 13:46:32 UTC
perm close


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