Skip to content

[24.0 backport] client: Client.postHijacked: use Client.buildRequest#45990

Merged
thaJeztah merged 2 commits intomoby:24.0from
thaJeztah:24.0_backport_hijack_share_request_builder
Jul 18, 2023
Merged

[24.0 backport] client: Client.postHijacked: use Client.buildRequest#45990
thaJeztah merged 2 commits intomoby:24.0from
thaJeztah:24.0_backport_hijack_share_request_builder

Conversation

@thaJeztah
Copy link
Member

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Just making my IDE and some linters slightly happier.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit e115552)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Use Client.buildRequest instead of a local copy of the same logic so
that we're using the same logic, and there's less chance of diverging.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit c219b09)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah marked this pull request as ready for review July 17, 2023 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api API kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants