Skip to content

Update "Allowed Users" in `tsh db ls" for databases with auto-user enabled#34671

Merged
greedy52 merged 3 commits intomasterfrom
STeve/33942_tsh_allowed_users
Nov 20, 2023
Merged

Update "Allowed Users" in `tsh db ls" for databases with auto-user enabled#34671
greedy52 merged 3 commits intomasterfrom
STeve/33942_tsh_allowed_users

Conversation

@greedy52
Copy link
Copy Markdown
Contributor

@greedy52 greedy52 commented Nov 16, 2023 •
edited by camscale
Loading

Fixes #33942

changelog: Fixed an issue "Allowed Users" in "tsh db ls" shows wrong user for databases with Automatic User Provisioning enabled.

$ tsh db ls
Name                       Description Allowed Users              Labels                   Connect 
-------------------------- ----------- -------------------------- ------------------------ ------- 
mongo-atlas                            [*]                                                         
self-hosted-mongo                      [*]                                                         
self-hosted-mongo-autouser             [STeve] (Auto-provisioned) EnableMongoAutoUser=true

@greedy52
Fix an issue "Allowed Users" in `tsh db ls" shows wrong user for
c099fc5 
databases with Automatic User Provisioning enabled.
@greedy52 greedy52 self-assigned this Nov 16, 2023
@greedy52 greedy52 changed the title Fix an issue "Allowed Users" in `tsh db ls" shows wrong user for databases with Automatic User Provisioning enabled Update "Allowed Users" in `tsh db ls" for databases with auto-user enabled Nov 16, 2023
@greedy52 greedy52 force-pushed the STeve/33942_tsh_allowed_users branch from 685052b to 1555325 Compare November 16, 2023 20:20
@greedy52 greedy52 added backport/branch/v13 tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Nov 16, 2023
@greedy52 greedy52 marked this pull request as ready for review November 16, 2023 20:20
@greedy52 greedy52 added the ux label Nov 16, 2023
@greedy52 greedy52 force-pushed the STeve/33942_tsh_allowed_users branch from 1555325 to ea0ddc9 Compare November 16, 2023 23:00
smallinsky
smallinsky approved these changes Nov 20, 2023
View reviewed changes
Comment thread lib/services/access_checker.go Outdated
Comment thread tool/tsh/common/tsh.go Outdated
@greedy52 greedy52 enabled auto-merge November 20, 2023 13:51
@greedy52 greedy52 added this pull request to the merge queue Nov 20, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 20, 2023
@greedy52 greedy52 added this pull request to the merge queue Nov 20, 2023
Merged via the queue into master with commit fd4e76c Nov 20, 2023
@greedy52 greedy52 deleted the STeve/33942_tsh_allowed_users branch November 20, 2023 15:02
@public-teleport-github-review-bot
Copy link
Copy Markdown

public-teleport-github-review-bot Bot commented Nov 20, 2023

@greedy52 See the table below for backport results.

Branch Result
branch/v13 Failed
branch/v14 Create PR

This was referenced Nov 21, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Tener Tener Tener approved these changes

@smallinsky smallinsky smallinsky approved these changes

Assignees

@greedy52 greedy52

Labels

size/md tsh tsh - Teleport's command line tool for logging into nodes running Teleport. ux

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allowed Users column in tsh db ls is inaccurate for auto-user provisioning databases

3 participants

@greedy52 @Tener @smallinsky