fix: build error in Node.js 22 caused by removed V8 AccessControl property - Githubissues

yfinkelstein / node-zookeeper

node.js client for Apache Zookeeper

MIT License

479 stars 111 forks source link

fix: build error in Node.js 22 caused by removed V8 AccessControl property #344

Closed DavidVujic closed 6 months ago

DavidVujic commented 6 months ago

Description

Replacing a removed V8 AccessControl enum property to the DEFAULT.

Details about the enum here: https://v8docs.nodesource.com/node-20.3/d2/dc3/namespacev8.html#a2e9a0288e6c2df2ecebc50944c3ff9fa

Motivation and Context

Fixing #343

How Has This Been Tested?

✅ CI ✅ Unit tests, integration tests, component tests

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have read the code of conduct.
[x] I have read the contributing guide.
[ ] I have updated the documentation accordingly (if applicable).