Skip to content

[v17] Sanitize aws service custom endpoints#49004

Merged
rosstimothy merged 1 commit intobranch/v17from
bot/backport-48992-branch/v17
Nov 14, 2024
Merged

[v17] Sanitize aws service custom endpoints#49004
rosstimothy merged 1 commit intobranch/v17from
bot/backport-48992-branch/v17

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #48992 to branch/v17

There is a slight behavior change in aws-sdk-go-v2 regarding custom
endpoints. The new sdk requires the endpoints to be a valid URI, and
the legacy sdk would allow a hostname only enpoint and automatically
apply a scheme. To prevent breaking existing configurations when
upgrading to v17, we now have to apply the same logic to endpoints
manually that the legacy sdk used to apply for us.

Closes #48760.
@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label Nov 14, 2024
@github-actions github-actions Bot added audit-log Issues related to Teleports Audit Log backport size/md labels Nov 14, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Nov 14, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 14, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Nov 14, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 14, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Nov 14, 2024
Merged via the queue into branch/v17 with commit 02970ee Nov 14, 2024
@rosstimothy rosstimothy deleted the bot/backport-48992-branch/v17 branch November 14, 2024 22:20
@r0mant r0mant mentioned this pull request Nov 15, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audit-log Issues related to Teleports Audit Log backport merge-for-v17 no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants