juju list-credentials --show-secrets does not display passwords

Bug #1564168 reported by Nicholas Skaggs
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Medium
Anastasia
2.6
Fix Released
Medium
Anastasia
juju-core (Ubuntu)
Fix Released
Undecided
Anastasia

Bug Description

juju list-credentials --show-secrets does not display password data, unless used with --format yaml.

The following commands all fail to show additonal information using --show-secrets
juju list-credentials --show-secrets
juju list-credentials --format json --show-secrets
juju list-credentials --format tabular --show-secrets

When used with --format yaml, --show-secrets works.

juju list-credentials --format yaml --show-secrets

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in juju-core (Ubuntu):
status: New → Confirmed
Changed in juju-core:
milestone: none → 2.0.0
status: New → Triaged
importance: Undecided → High
tags: added: docteam juju-release-support
tags: added: usability
Revision history for this message
Anastasia (anastasia-macmood) wrote :

This is working as designed - password data can only be seen in yaml format.
When --show-secrets is specified for all other formats, we need to inform the user to that effect:
"--show-secrets displays password data in yaml format only"

Changed in juju-core:
importance: High → Medium
affects: juju-core → juju
Changed in juju:
milestone: 2.0.0 → none
milestone: none → 2.0.0
Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.0.0 → 2.0.1
Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.0.1 → none
Changed in juju:
status: Triaged → In Progress
assignee: nobody → Anastasia (anastasia-macmood)
Revision history for this message
Anastasia (anastasia-macmood) wrote :

Actually, both json and yaml formats display secrets when asked. I'll update option and command help.

Revision history for this message
Anastasia (anastasia-macmood) wrote :
Changed in juju-core (Ubuntu):
status: Confirmed → Fix Committed
assignee: nobody → Anastasia (anastasia-macmood)
Revision history for this message
Anastasia (anastasia-macmood) wrote :

Merge PR brining this change into develop among other things: https://github.com/juju/juju/pull/10390

Changed in juju:
status: In Progress → Fix Committed
Changed in juju:
status: Fix Committed → Fix Released
Changed in juju-core (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.