Skip to content

fix local re-register#46087

Merged
fspmarshall merged 1 commit intomasterfrom
fspmarshall/fix-local-reregister
Sep 3, 2024
Merged

fix local re-register#46087
fspmarshall merged 1 commit intomasterfrom
fspmarshall/fix-local-reregister

Conversation

@fspmarshall
Copy link
Copy Markdown
Contributor

Fixes an issue introduced in #41467 which could cause reissue of certificates to fail for a service which was running alongside a local auth server, had a listener bound to 0.0.0.0, and did not have advertise_ip set. With this change, local re-register takes on the same behavior as local register, using 127.0.0.1 as the default remote address value.

Fixes #43856

changelog: fixed an issue that would cause reissue of certificates to fail in some scenarios where a local auth service was present.

Comment thread lib/service/connect.go Outdated
Comment thread lib/service/connect.go Outdated
@fspmarshall fspmarshall force-pushed the fspmarshall/fix-local-reregister branch from 210043c to cf9b9fc Compare September 3, 2024 16:38
@fspmarshall fspmarshall added this pull request to the merge queue Sep 3, 2024
Merged via the queue into master with commit 440f586 Sep 3, 2024
@fspmarshall fspmarshall deleted the fspmarshall/fix-local-reregister branch September 3, 2024 17:22
@public-teleport-github-review-bot
Copy link
Copy Markdown

@fspmarshall See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Changing a cluster's proxy_service.public_addr with no https_keypairs results in rotation state sync errors

3 participants