Project

General

Profile

Actions

Bug #41320

closed

Feature #47765: mgr/dashboard: security improvements

mgr/dashboard: passwords and other sensitive information is written to logs

Added by Ernesto Puerta over 4 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
General
Target version:
-
% Done:

0%

Source:
Tags:
security
Backport:
nautilus
Regression:
No
Severity:
2 - major
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Currently dashboard is storing in plain text logs the following sentitive information:

- Dashboard user names, passwords and roles. -> handled in #37503

log_channel(audit) log [DBG] : from='client.4126 -' entity='client.admin' cmd=[{"username": "admin", "rolename": "administrator", "prefix": "dashboard ac-user-create", "password": "admin"}]

- RGW API keys:

cmd=[{"prefix": "dashboard set-rgw-api-access-key", "target": ["mgr", ""], "value": "<real_key>"}]
cmd=[{"prefix": "dashboard set-rgw-api-secret-key", "target": ["mgr", ""], "value": "<real_key>"}]:

- JWT tokens:

"JWT Token: <real_token>"

This information should be redacted from the logs. While access to logs could be limited to privileged users, this is considered insecure (even with hashed passwords).


Related issues 2 (0 open2 closed)

Related to Ceph - Bug #37503: Audit log: mgr module passwords set on CLI written as plaintext in log filesResolvedNeha Ojha

Actions
Copied to Dashboard - Backport #41980: nautilus: mgr/dashboard: passwords and other sensitive information is written to logsResolvedNathan CutlerActions
Actions #1

Updated by Kefu Chai over 4 years ago

  • Assignee set to Kefu Chai
Actions #2

Updated by Kefu Chai over 4 years ago

  • Assignee deleted (Kefu Chai)
Actions #4

Updated by Sebastian Wagner over 4 years ago

  • Related to Bug #37503: Audit log: mgr module passwords set on CLI written as plaintext in log files added
Actions #5

Updated by Sebastian Wagner over 4 years ago

  • Description updated (diff)
  • Pull request ID set to 30445

Let's handle the autit log in #37503 . This way we can make https://github.com/ceph/ceph/pull/30445 a real PR for this issue here.

Actions #6

Updated by Kefu Chai over 4 years ago

  • Status changed from New to Resolved
Actions #7

Updated by Lenz Grimmer over 4 years ago

  • Status changed from Resolved to Pending Backport
  • Backport set to nautilus

Setting this to "pending backport", as this should be fixed for Nautilus, too.

Actions #8

Updated by Nathan Cutler over 4 years ago

  • Copied to Backport #41980: nautilus: mgr/dashboard: passwords and other sensitive information is written to logs added
Actions #9

Updated by Lenz Grimmer over 4 years ago

  • Assignee set to Kefu Chai
Actions #10

Updated by Nathan Cutler over 4 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions #11

Updated by Ernesto Puerta over 3 years ago

  • Parent task set to #47765
Actions #12

Updated by Ernesto Puerta about 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 132 to General
Actions

Also available in: Atom PDF