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

fix: node-ffi-napi-compatibility #1821

Merged

Conversation

genaris
Copy link
Contributor

@genaris genaris commented Apr 5, 2024

Trying to re-gain compatibliity with latest node versions (18.20, 20.12, 21.6) by using a patched version of @2060.io/ffi-napi. The issue was caused by an experimental feature in N-API (see this commit), which was enabled when compiling ffi-napi because it had NAPI_EXPERIMENTAL flag set (originally this was needed until the release of node 12.17, but the flag wasn't removed since the repo was not being maintained after that).

Given that @hyperledger/aries-askar-nodejs, @hyperledger/indy-vdr-nodejs and @hyperledger/anoncreds had pinned previous version of ffi-napi, I think we'll need to do a new minor release of those modules before merging this, since at the moment we need to pin this new version in resolutions.

genaris added 3 commits April 5, 2024 15:00
Signed-off-by: Ariel Gentile <[email protected]>
Signed-off-by: Ariel Gentile <[email protected]>
@genaris genaris marked this pull request as draft April 5, 2024 18:26
Signed-off-by: Ariel Gentile <[email protected]>
@genaris genaris marked this pull request as ready for review April 11, 2024 11:15
@genaris genaris enabled auto-merge (squash) April 11, 2024 12:05
@genaris genaris merged commit 81d351b into openwallet-foundation:main Apr 12, 2024
12 checks passed
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