[SRU] ceilometer writing snmp credentials to log file
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ceilometer |
In Progress
|
Undecided
|
Edward Hope-Morley | ||
Ubuntu Cloud Archive |
Fix Released
|
High
|
Unassigned | ||
Ocata |
Triaged
|
High
|
Unassigned | ||
Pike |
Triaged
|
High
|
Unassigned | ||
Queens |
Fix Released
|
High
|
Unassigned | ||
Rocky |
Fix Released
|
High
|
Unassigned | ||
Stein |
Fix Released
|
High
|
Unassigned | ||
ceilometer (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Bionic |
Fix Released
|
High
|
Unassigned | ||
Cosmic |
Fix Released
|
High
|
Unassigned | ||
Disco |
Fix Released
|
High
|
Unassigned |
Bug Description
[Impact]
This SRU proposal is to patch the Ubuntu ceilometer package so that the ceilometer-agent switches printing the contents of polling.yaml from INFO to DEBUG. This is mostly an interim fix to make it easy to stop the presence of sensitive data in the ceilometer logfiles when DEBUG logging is not activated. Another bug will be raised to propose sanitising the data printed.
[Test Case]
* deploy Openstack Q/R/S with ceilometer
* enable debug logging
* check that /var/log/
2019-01-09 11:40:50.641 25495 DEBUG ceilometer.agent [-] Config file: {'sources': [{'interval': 300, 'meters'...
i.e. ensure that the log is printed using DEBUG (not INFO)
[Regression Potential]
Users with debug mode disabled will no longer see this line.
----
The ceilometer-
This presents a security risk if e.g. resources contain sensitive information like when specifying snmp targets with the url containing the username, password etc.
There are a couple of ways we could solve this, namely; (1) don't log this info at all, (2) sanitise the contents prior to logging as DEBUG (3) switch to using config for the snmp credentials in a similar way to how the Triple0Discoverer does it [2] - this would only support having the same creds everywhere thought which may not be desirable.
[1] https:/
[2] https:/
CVE References
Changed in ceilometer: | |
assignee: | nobody → Edward Hope-Morley (hopem) |
tags: | added: sts sts-sru-needed |
description: | updated |
Changed in ceilometer (Ubuntu Bionic): | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in ceilometer (Ubuntu Cosmic): | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in ceilometer (Ubuntu Disco): | |
importance: | Undecided → High |
status: | New → Triaged |
information type: | Public → Private Security |
tags: |
added: sts-sru-done removed: sts-sru-needed |
Fix proposed to branch: master /review. openstack. org/629891
Review: https:/