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.

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

Update wiremock and switch from multipart to multer #31

Merged
merged 2 commits into from
Feb 28, 2024

Conversation

sjoerdsimons
Copy link
Collaborator

Bump wiremock to 0.6 and replace multipart with multer as multipart is unmaintained

@sjoerdsimons sjoerdsimons requested a review from refi64 February 18, 2024 19:06
gitlab-runner-mock/src/api/artifacts.rs Outdated Show resolved Hide resolved
gitlab-runner-mock/src/api/artifacts.rs Outdated Show resolved Hide resolved
@sjoerdsimons sjoerdsimons force-pushed the sjoerd/update-dependencies branch from 637c4d3 to 252b311 Compare February 28, 2024 19:00
Most of the changes are due to wiremock moving from http_types to the
http crate. While there also change all return codes to use StatusCode
values to make it more readable.

Signed-off-by: Sjoerd Simons <sjoerd@collabora.com>
The multipart crate has been unmaintained for a while, replace with
multer which e.g. also used by axum. Unfortunately multer is async only,
while the wiremock Response trait is sync[0]. So that does require some
hoops to jump through for now.

0: LukeMathWalker/wiremock-rs#84

Signed-off-by: Sjoerd Simons <sjoerd@collabora.com>
@sjoerdsimons sjoerdsimons force-pushed the sjoerd/update-dependencies branch from 252b311 to 86e4855 Compare February 28, 2024 19:07
@sjoerdsimons sjoerdsimons requested a review from refi64 February 28, 2024 19:07
@refi64 refi64 added this pull request to the merge queue Feb 28, 2024
Merged via the queue into main with commit 8102e50 Feb 28, 2024
14 checks passed
@sjoerdsimons sjoerdsimons deleted the sjoerd/update-dependencies branch February 29, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants