Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[v17] Frontend for multi-port TCP app access #49989

Merged
merged 2 commits into from
Dec 17, 2024
Merged

[v17] Frontend for multi-port TCP app access #49989

merged 2 commits into from
Dec 17, 2024

Conversation

ravicious
Copy link
Member

@ravicious ravicious commented Dec 10, 2024 •
edited
Loading

@ravicious ravicious added the no-changelog Indicates that a PR does not require a changelog entry label Dec 10, 2024
@ravicious ravicious marked this pull request as ready for review December 11, 2024 17:17
@github-actions github-actions bot added application-access size/md tsh tsh - Teleport's command line tool for logging into nodes running Teleport. ui labels Dec 11, 2024
@github-actions github-actions bot requested review from avatus and kiosion December 11, 2024 17:18
@ravicious ravicious requested review from gzdunek and removed request for kiosion December 11, 2024 17:23
@ravicious ravicious force-pushed the r7s/v17/backport-49921-windows-vnet-restructure branch from 66b43fb to cf1e476 Compare December 17, 2024 14:55
Base automatically changed from r7s/v17/backport-49921-windows-vnet-restructure to branch/v17 December 17, 2024 15:34
@ravicious
Surface target ports in tsh and Connect (#49688)
670a34a 
* tsh: Print target ports when returning multi-port apps

* Add TCP ports to App message in lib/teleterm

* Connect: Add port selector for multi-port TCP apps

* Add controls to ActionButtons story

* Add control for rendering lots of menu items in ActionButtons story

* Include target port in VNet modals and notifications in Connect

* Empty commit to trigger CI
@ravicious
VNet: Refuse conn if local port doesn't match app spec
7fd6be8
@ravicious ravicious force-pushed the r7s/v17/multi-port-ui branch from f67d887 to 7fd6be8 Compare December 17, 2024 15:44
@ravicious ravicious enabled auto-merge December 17, 2024 15:47
@ravicious ravicious added this pull request to the merge queue Dec 17, 2024
Merged via the queue into branch/v17 with commit 6d5d673 Dec 17, 2024
44 checks passed
@ravicious ravicious deleted the r7s/v17/multi-port-ui branch December 17, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gzdunek gzdunek gzdunek approved these changes

@avatus avatus avatus approved these changes

Assignees
No one assigned
Labels
application-access no-changelog Indicates that a PR does not require a changelog entry size/md tsh tsh - Teleport's command line tool for logging into nodes running Teleport. ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ravicious @avatus @gzdunek