Skip to content

Conversation

riyaz-ali
Copy link

This PR adds support for compile-time authorization as mentioned in #560 🚀

Would ❤️ to see support for it being included in the upstream, as currently, AFAIK 🤔, no mainstream node.js package for sqlite3 provides support for this functionality.

This commit adds support for sqlite3_set_authorizer function
by implementing AuthorizerFunction and the related database->setAuthorizer
method.
@riyaz-ali
Copy link
Author

P.S. This is my first attempt at building node.js addons and working with C++ in general 😅 would love to receive constructive feedback and advice 🙌 , if possible.

@etienne-dldc
Copy link

Hi 👋 any update on this ? 😇

@riyaz-ali riyaz-ali requested a review from JoshuaWise as a code owner April 7, 2022 04:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants