Skip to content

[v12] Fix underlying reader not a terminal issues#25242

Merged
Joerger merged 1 commit intobranch/v12from
joerger/v12/catch-underlying-reader-not-a-terminal-error
Apr 27, 2023
Merged

[v12] Fix underlying reader not a terminal issues#25242
Joerger merged 1 commit intobranch/v12from
joerger/v12/catch-underlying-reader-not-a-terminal-error

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Apr 27, 2023

Backport #25102 to branch/v12

@github-actions github-actions Bot added backport size/md tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Apr 27, 2023
@github-actions github-actions Bot requested review from probakowski, r0mant and zmb3 April 27, 2023 00:56
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from probakowski April 27, 2023 01:35
@Joerger Joerger enabled auto-merge April 27, 2023 17:20
* * Use Stderr during headless login to fix rsync compatibility

* Improve error messages when attempting to login without a terminal

* Skip MOTD acknowledement when logging in without a terminal (headless,
  sso)

* When attempting re-login without a terminal, make a debug log and
  return the original error

* Return access denied error on unexpected handshake failures in tsh
  proxy ssh

* Fix TestProxySSH test and add it to the flaky test detector testsToSkip
list.
@Joerger Joerger force-pushed the joerger/v12/catch-underlying-reader-not-a-terminal-error branch from 8e1702f to b021c0e Compare April 27, 2023 17:22
@Joerger Joerger added this pull request to the merge queue Apr 27, 2023
Merged via the queue into branch/v12 with commit 5c97582 Apr 27, 2023
@Joerger Joerger deleted the joerger/v12/catch-underlying-reader-not-a-terminal-error branch April 27, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/md 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