Skip to content

Headless Login explicit username#24689

Merged
Joerger merged 4 commits intomasterfrom
joerger/headless-explicit-username
Apr 24, 2023
Merged

Headless Login explicit username#24689
Joerger merged 4 commits intomasterfrom
joerger/headless-explicit-username

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Apr 17, 2023

Return an error if user is not explicitly set for headless login. This should help prevent users from hitting #23436 due to their OS user being used for login by default.

@Joerger Joerger requested a review from rosstimothy April 17, 2023 18:54
@github-actions github-actions Bot added size/sm tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Apr 17, 2023
@github-actions github-actions Bot requested review from rudream and smallinsky April 17, 2023 18:55
Copy link
Copy Markdown
Collaborator

@zmb3 zmb3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about a test?

@Joerger
Copy link
Copy Markdown
Contributor Author

Joerger commented Apr 18, 2023

@zmb3 I added a test, let me know what you think.

@rosstimothy maybe you can reuse / add to this test for #24701

Comment thread lib/auth/auth.go
Comment thread lib/client/api.go Outdated
Comment thread lib/auth/auth.go Outdated
@Joerger Joerger force-pushed the joerger/headless-explicit-username branch from 0068b19 to fe9a190 Compare April 24, 2023 17:17
@Joerger Joerger force-pushed the joerger/headless-explicit-username branch from aab169f to 69ffe5d Compare April 24, 2023 17:39
@Joerger Joerger enabled auto-merge April 24, 2023 17:51
@Joerger Joerger added this pull request to the merge queue Apr 24, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 24, 2023
@Joerger Joerger added this pull request to the merge queue Apr 24, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 24, 2023
@Joerger Joerger added this pull request to the merge queue Apr 24, 2023
Merged via the queue into master with commit 40ba8f3 Apr 24, 2023
@Joerger Joerger deleted the joerger/headless-explicit-username branch April 24, 2023 19:50
@public-teleport-github-review-bot
Copy link
Copy Markdown

@Joerger See the table below for backport results.

Branch Result
branch/v12 Failed
branch/v13 Failed

Joerger added a commit that referenced this pull request Apr 24, 2023
* Return an error if user is not explicity set for headless login.

* Add test.

* Resolve comments.

* Fix typo.
Joerger added a commit that referenced this pull request Apr 24, 2023
* Return an error if user is not explicity set for headless login.

* Add test.

* Resolve comments.

* Fix typo.
Joerger added a commit that referenced this pull request Apr 25, 2023
* Return an error if user is not explicity set for headless login.

* Add test.

* Resolve comments.

* Fix typo.
Joerger added a commit that referenced this pull request Apr 25, 2023
* Return an error if user is not explicity set for headless login.

* Add test.

* Resolve comments.

* Fix typo.
@r0mant r0mant mentioned this pull request Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants