Skip to content

feat(ffi): upgrade mock server to v2#509

Merged
JP-Ellis merged 2 commits intomasterfrom
feat/upgrade-mock-server-to-v2
Dec 17, 2025
Merged

feat(ffi): upgrade mock server to v2#509
JP-Ellis merged 2 commits intomasterfrom
feat/upgrade-mock-server-to-v2

Conversation

@JP-Ellis
Copy link
Contributor

Upgrade the mock server dependency within the FFI crate to version 2.

In general, this should be backwards compatible, and should bring through to the FFI a few fixes.

@JP-Ellis JP-Ellis self-assigned this Dec 15, 2025
@JP-Ellis JP-Ellis changed the title chore(ffi): all async fns in ffi_fn macro feat(ffi): upgrade mock server to v2 Dec 15, 2025
This wraps the inner asynchronous function with a
`::futures::executor::block_on`, allowing for asynchronous function to
be more easily called by the FFI.

Signed-off-by: JP-Ellis <[email protected]>
@JP-Ellis JP-Ellis force-pushed the feat/upgrade-mock-server-to-v2 branch from 77a37db to b9c8719 Compare December 15, 2025 23:28
@rholshausen
Copy link
Contributor

No idea why the builds are failing. I tried rerunning them, but it might be they are taking too long on the MacOS agents that the agents are being killed. The v2-engine might be some regression, but your changes should not be affecting that.

@JP-Ellis
Copy link
Contributor Author

My guess is there's some sort of new dependency required to build the FFI, that the linker can't find when building the C example. I'll suss it out.

The v2 mock server has a number of improvements over v1. This should be
generally backwards compatible.

Signed-off-by: JP-Ellis <[email protected]>
@JP-Ellis JP-Ellis force-pushed the feat/upgrade-mock-server-to-v2 branch from b9c8719 to d1b63ff Compare December 17, 2025 06:24
@JP-Ellis JP-Ellis merged commit 9d1a12a into master Dec 17, 2025
35 of 36 checks passed
@JP-Ellis JP-Ellis deleted the feat/upgrade-mock-server-to-v2 branch December 17, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants