Bug 1227019 - Require sos >= 3.3 when available - ovirt sosreport plugin doesn't obfuscate password used in aaa extensions
Summary: Require sos >= 3.3 when available - ovirt sosreport plugin doesn't obfuscate ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-log-collector
Classification: oVirt
Component: General
Version: 4.1.0
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ovirt-4.1.2
: 4.1.0
Assignee: Lev Veyde
QA Contact: Aleksei Slaikovskii
URL:
Whiteboard:
Depends On: 1293044 1411314
Blocks: 1444461
TreeView+ depends on / blocked
 
Reported: 2015-06-01 17:37 UTC by Tim Speetjens
Modified: 2019-04-28 13:28 UTC (History)
12 users (show)

Fixed In Version: ovirt-log-collector-4.1.1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-05-23 08:16:37 UTC
oVirt Team: Integration
Embargoed:
ykaul: needinfo-
rule-engine: ovirt-4.1+
ylavi: exception+
ylavi: planning_ack+
sbonazzo: devel_ack+
pstehlik: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 41809 0 master ABANDONED sos: obfuscate passwords specified in aaa profiles Never
oVirt gerrit 75128 0 master MERGED Updated requirement to require sos >= 3.3 2017-04-04 09:57:38 UTC
oVirt gerrit 75130 0 ovirt-log-collector-4.1 MERGED Updated requirement to require sos >= 3.3 2017-04-04 10:03:17 UTC

Description Tim Speetjens 2015-06-01 17:37:58 UTC
Description of problem:
oVirt plugin for sosreport includes /etc/ovirt-engine/aaa directory, which commonly includes bind-dn and password

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

How reproducible:
100%

Steps to Reproduce:
1. Specify a password in /etc/ovirt-engine/aaa/<anyprofile>.properties
2. Run a sosreport


Actual results:
The properties file for the profile contains the password, used for binding to the directory

Expected results:
Passwords should be replaced by ******

Additional info:

Comment 1 Sandro Bonazzola 2015-06-09 06:18:24 UTC
Please open a corresponding ticket on sos report: https://github.com/sosreport/sos
and push the patch there too. Starting with EL6.7 and already on Fedora and EL7+ log-collector is using sos plugins only.

Comment 2 Bryn M. Reeves 2015-06-25 13:49:56 UTC
Please when opening upstream issues please include enough information that the reader can tell what is being reported (and ideally fix it) without having to visit another site (i.e. everything that's in comment#0 not "See this rhbz#").

Comment 3 Tim Speetjens 2015-07-22 07:15:51 UTC
Issue reported at https://github.com/sosreport/sos/issues/588

Comment 4 Sandro Bonazzola 2015-09-02 10:10:32 UTC
Closing upstream since ovirt sos report is handled in sos upstream project.
Please open a BZ on RHEL (And Fedora for upstream) sos component for including the submitted patch.

Comment 5 Yaniv Lavi 2015-09-03 09:22:58 UTC
This should remain open as a tracker for the fix in SOS. Please add the dependency once that bubg is created.

Comment 6 Sandro Bonazzola 2015-09-03 15:01:02 UTC
(In reply to Yaniv Dary from comment #5)
> This should remain open as a tracker for the fix in SOS. Please add the
> dependency once that bubg is created.

I don't see why a bug in a sos plugin not shipped within log-collector should be tracked in log-collector.

We can require a newer version of sos if you want, but this is not what this bug is about.

Comment 7 Bryn M. Reeves 2015-09-04 13:22:34 UTC
FYI we'll be attempting to get a sos-3.3 release out fairly soon (with this patch) and will update Fedora packages once that is out.

Comment 8 Bryn M. Reeves 2015-09-04 13:24:27 UTC
Upstream: https://github.com/sosreport/sos/commit/da3f566

Comment 9 Sandro Bonazzola 2015-09-04 14:22:31 UTC
Thanks Bryn!

Comment 10 Yaniv Lavi 2015-09-09 15:55:27 UTC
Why did you remove the blocker?

Comment 11 Sandro Bonazzola 2015-09-11 15:42:36 UTC
Why should this be a blocker?
We need to require a package which is not even released upstream and which is not yet in any supported distribution.
If we block on this we need to postpone oVirt 3.6.0 until sos-3.3 will be out in Fedora and CentOS.

Comment 12 Sandro Bonazzola 2015-09-29 08:52:37 UTC
Devel NAK: Waiting on upstream sos to release 3.3.

Comment 13 Red Hat Bugzilla Rules Engine 2015-10-19 11:03:17 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 14 Yaniv Lavi 2015-10-29 12:16:30 UTC
In oVirt testing is done on single release by default. Therefore I'm removing the 4.0 flag. If you think this bug must be tested in 4.0 as well, please re-add the flag. Please note we might not have testing resources to handle the 4.0 clone.

Comment 15 Bryn M. Reeves 2015-12-18 12:36:32 UTC
We're hoping to release sos-3.3 next week. I'll update the bug again with a link to the release once it's out.

Comment 16 Sandro Bonazzola 2016-05-02 10:05:04 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 17 Yaniv Lavi 2016-05-23 13:19:42 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 18 Yaniv Lavi 2016-05-23 13:23:27 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 19 Yaniv Lavi 2016-06-02 08:07:27 UTC
Do you know when this version will be released?

Comment 20 Bryn M. Reeves 2016-06-02 09:47:23 UTC
We're hoping to tag a beta commit today with a view to 3.3 being release mid next week.

Comment 21 Yaniv Lavi 2016-06-02 13:58:52 UTC
(In reply to Bryn M. Reeves from comment #20)
> We're hoping to tag a beta commit today with a view to 3.3 being release mid
> next week.

What is the ETA for release in RHEL? 7.3?

Comment 22 Bryn M. Reeves 2016-06-02 14:10:41 UTC
bug 1293044.

Please speak to Pavel Moravec (pmoravec) for specific RHEL7 package queries - he's been the RHEL7 maintainer for a couple of releases now.

Comment 23 Yaniv Lavi 2016-06-02 14:15:22 UTC
Can you please provide the info request in comment #21?

Comment 25 Sandro Bonazzola 2016-07-05 06:01:42 UTC
Upstream SOS 3.3 has been released.
- check if already available in Fedora
- check if already available in EPEL / Virt SIG
- check if it works out of the box with current log-collector code. If not, please open a rebase bug in order to support it.

Comment 26 Bryn M. Reeves 2016-07-05 10:59:57 UTC
I'm hoping to have a rawhide build done late this week or early next, with updates to follow for other supported Fedora releases.

Comment 27 Sandro Bonazzola 2016-12-05 10:56:39 UTC
(In reply to Bryn M. Reeves from comment #26)
> I'm hoping to have a rawhide build done late this week or early next, with
> updates to follow for other supported Fedora releases.

Bryn, any updates?

Comment 30 Sandro Bonazzola 2017-04-04 06:40:37 UTC
Lev, 3.3 is now within RHEL and 3.4 is now available in Fedora.
Can you please update log-collector requires on 3.3 and check data within the report to match what asked in comment #0 ?

Comment 31 Lev Veyde 2017-04-04 10:59:53 UTC
(In reply to Sandro Bonazzola from comment #30)
> Lev, 3.3 is now within RHEL and 3.4 is now available in Fedora.
> Can you please update log-collector requires on 3.3 and check data within
> the report to match what asked in comment #0 ?

Done, fixed in ovirt-log-collector-4.1.1.

Verified:
$ cat internal.properties 
config.datasource.jdbcurl=jdbc:postgresql://localhost:5432/engine?sslfactory=org.postgresql.ssl.NonValidatingFactory
config.datasource.dbuser=engine
config.datasource.dbpassword=*********
config.datasource.jdbcdriver=org.postgresql.Driver

Comment 34 Aleksei Slaikovskii 2017-05-10 11:42:00 UTC
Verified on ovirt-log-collector-4.1.1-1.el7ev.noarch

cat ./sosreport-engine-20170510142909/etc/ovirt-engine/aaa/internal.properties
config.datasource.jdbcurl=jdbc:postgresql://localhost:5432/engine?sslfactory=org.postgresql.ssl.NonValidatingFactory
config.datasource.dbuser=engine
config.datasource.dbpassword=*********
config.datasource.jdbcdriver=org.postgresql.Driver
config.datasource.schemaname=aaa_jdbc


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