Skip to content

sqlite: create authorization api #59928

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.

Sign up for GitHub

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

Closed
araujogui wants to merge 18 commits into from
Closed

sqlite: create authorization api #59928

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
6bf9173
sqlite: create authz method
araujogui Sep 18, 2025
bf0fc42
sqlite: improve error handling
araujogui Sep 18, 2025
41717ac
doc: create setauthorizer method doc
araujogui Sep 18, 2025
60ad323
doc: lint
araujogui Sep 18, 2025
19cdadf
doc: update doc link
araujogui Sep 18, 2025
e25f0ce
sqlite: remove duplicated proto init
araujogui Sep 18, 2025
e7288f3
sqlite: break long lines
araujogui Sep 18, 2025
3662e71
sqlite: format code
araujogui Sep 18, 2025
f63cba7
sqlite: store callback in internal field
araujogui Sep 22, 2025
2714896
sqlite: restrict valid authz code
araujogui Sep 23, 2025
e66a64c
sqlite: improve authorizer error handling
araujogui Sep 23, 2025
9e96632
sqlite: store authz error in internal object
araujogui Sep 24, 2025
a1553ef
sqlite: refactor authz code validation logic
araujogui Sep 24, 2025
e6ab93d
sqlite: refactor ToLocalChecked usage
araujogui Sep 25, 2025
254680d
sqlite: create constants doc
araujogui Sep 25, 2025
b51043c
sqlite: create sqlite_ignore test cases
araujogui Sep 25, 2025
526216d
sqlite: refactor authz error handling
araujogui Sep 26, 2025
0f4816f
sqlite: fix format
araujogui Sep 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
sqlite: format code
  • Loading branch information
@araujogui
araujogui committed Sep 18, 2025
commit 3662e7187f22182404b01f210eb06715eca7eabf
28 changes: 12 additions & 16 deletions src/node_sqlite.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -689,17 +689,17 @@ int AuthorizerFunction::xAuthorizer(void* user_data,
// Convert SQLite authorizer parameters to JavaScript values
js_argv.emplace_back(Integer::New(isolate, action_code));
js_argv.emplace_back(
NullableSQLiteStringToValue(isolate, param1).ToLocalChecked());
NullableSQLiteStringToValue(isolate, param1).ToLocalChecked());
js_argv.emplace_back(
NullableSQLiteStringToValue(isolate, param2).ToLocalChecked());
NullableSQLiteStringToValue(isolate, param2).ToLocalChecked());
js_argv.emplace_back(
NullableSQLiteStringToValue(isolate, param3).ToLocalChecked());
NullableSQLiteStringToValue(isolate, param3).ToLocalChecked());
js_argv.emplace_back(
NullableSQLiteStringToValue(isolate, param4).ToLocalChecked());
NullableSQLiteStringToValue(isolate, param4).ToLocalChecked());

TryCatch try_catch(isolate);
MaybeLocal<Value> retval = fn->Call(
context, Undefined(isolate), js_argv.size(), js_argv.data());
MaybeLocal<Value> retval =
fn->Call(context, Undefined(isolate), js_argv.size(), js_argv.data());

if (try_catch.HasCaught()) {
// If there's an exception in the callback, deny the operation
Expand Down Expand Up @@ -1926,25 +1926,23 @@ void DatabaseSync::SetAuthorizer(const FunctionCallbackInfo<Value>& args) {
Environment* env = Environment::GetCurrent(args);
Isolate* isolate = env->isolate();

if ( args[0]->IsNull() ) {
if (args[0]->IsNull()) {
// Clear the authorizer
sqlite3_set_authorizer(db->connection_, nullptr, nullptr);
return;
}

if (!args[0]->IsFunction()) {
THROW_ERR_INVALID_ARG_TYPE(
isolate,
"The \"callback\" argument must be a function or null.");
isolate, "The \"callback\" argument must be a function or null.");
return;
}

Local<Function> fn = args[0].As<Function>();
AuthorizerFunction* user_data = new AuthorizerFunction(env, fn, db);

int r = sqlite3_set_authorizer(db->connection_,
AuthorizerFunction::xAuthorizer,
user_data);
int r = sqlite3_set_authorizer(
db->connection_, AuthorizerFunction::xAuthorizer, user_data);

if (r != SQLITE_OK) {
delete user_data;
Expand Down Expand Up @@ -3264,10 +3262,8 @@ static void Initialize(Local<Object> target,
DatabaseSync::EnableLoadExtension);
SetProtoMethod(
isolate, db_tmpl, "loadExtension", DatabaseSync::LoadExtension);
SetProtoMethod(isolate,
db_tmpl,
"setAuthorizer",
DatabaseSync::SetAuthorizer);
SetProtoMethod(
isolate, db_tmpl, "setAuthorizer", DatabaseSync::SetAuthorizer);
SetSideEffectFreeGetter(isolate,
db_tmpl,
FIXED_ONE_BYTE_STRING(isolate, "isOpen"),
Expand Down
Loading