Skip to content

[v17] kube: prevent init failure when proxy listener addr host is unspecified#51864

Merged
tigrato merged 2 commits intobranch/v17from
bot/backport-51839-branch/v17
Feb 5, 2025
Merged

[v17] kube: prevent init failure when proxy listener addr host is unspecified#51864
tigrato merged 2 commits intobranch/v17from
bot/backport-51839-branch/v17

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Feb 5, 2025

Backport #51839 to branch/v17

changelog: Prevent Teleport proxy failing to initialize when listener address's host component is empty.

tigrato and others added 2 commits February 4, 2025 20:28
This PR fixes an issue caused by configuring Teleport proxy listener
address without a host `:443` or in ipv6 form `[::]:443`.

The call to `utils.SplitHostPort` was checking if the host was empty
after the split which caused proxies to fail to initialize.

`:443` is equivalent to `0.0.0.0:443`.

Fixes #51838

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
Co-authored-by: Zac Bergquist <zac.bergquist@goteleport.com>
@tigrato tigrato enabled auto-merge February 5, 2025 15:47
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from timothyb89 February 5, 2025 16:04
@tigrato tigrato added this pull request to the merge queue Feb 5, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 5, 2025
@tigrato tigrato added this pull request to the merge queue Feb 5, 2025
Merged via the queue into branch/v17 with commit 3073309 Feb 5, 2025
@tigrato tigrato deleted the bot/backport-51839-branch/v17 branch February 5, 2025 16:46
@camscale camscale mentioned this pull request Feb 13, 2025
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