mirror of https://github.com/status-im/consul.git
Fix panic on acl token read with -self and -expanded
This commit is contained in:
parent
65b0faf96b
commit
927033e672
|
@ -0,0 +1,3 @@
|
|||
```release-note:bug
|
||||
cli: when `acl token read` is used with the `-self` and `-expanded` flags, return an error instead of panicking
|
||||
```
|
|
@ -92,6 +92,13 @@ func (c *cmd) Run(args []string) int {
|
|||
return 1
|
||||
}
|
||||
} else {
|
||||
// TODO: consider updating this CLI command and underlying HTTP API endpoint
|
||||
// to support expanded read of a "self" token, which is a much better user workflow.
|
||||
if c.expanded {
|
||||
c.UI.Error("Cannot use both -expanded and -self. Instead, use -expanded and -id=<accessor id>.")
|
||||
return 1
|
||||
}
|
||||
|
||||
t, _, err = client.ACL().TokenReadSelf(nil)
|
||||
if err != nil {
|
||||
c.UI.Error(fmt.Sprintf("Error reading token: %v", err))
|
||||
|
|
Loading…
Reference in New Issue