Skip to content

[v11] Fix port forwarding when using a label based target#26704

Merged
rosstimothy merged 1 commit intobranch/v11from
tross/backport-26394/v11
May 22, 2023
Merged

[v11] Fix port forwarding when using a label based target#26704
rosstimothy merged 1 commit intobranch/v11from
tross/backport-26394/v11

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #26394 to branch/v11

Label based connections that do not provide a command and open a
shell for the first match were only performing a portion of the logic
that non label based interactive sessions were. Instead of connecting
directly to a host and opening a session, `tc.runShellOrCommandOnSingleNode`
is called to achieve the same behavior as non label based connections.

In addition to fixing #23261 this should also add support for `-N` in
the same scenario.
@rosstimothy rosstimothy force-pushed the tross/backport-26394/v11 branch from 5f9a2d3 to 0cc3a84 Compare May 22, 2023 16:55
@rosstimothy rosstimothy marked this pull request as ready for review May 22, 2023 17:54
@github-actions github-actions Bot requested review from kimlisa and r0mant May 22, 2023 17:54
@rosstimothy rosstimothy enabled auto-merge May 22, 2023 19:19
@rosstimothy rosstimothy added this pull request to the merge queue May 22, 2023
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from r0mant May 22, 2023 20:19
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 22, 2023
@rosstimothy rosstimothy added this pull request to the merge queue May 22, 2023
Merged via the queue into branch/v11 with commit e8a8798 May 22, 2023
@rosstimothy rosstimothy deleted the tross/backport-26394/v11 branch May 22, 2023 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants