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

Show file tree
Hide file tree
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: refactor authz code validation logic
  • Loading branch information
@araujogui
araujogui committed Sep 24, 2025
commit a1553ef9d190f1e147b756be0f84d03450926d66
13 changes: 2 additions & 11 deletions src/node_sqlite.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1967,8 +1967,8 @@ int DatabaseSync::AuthorizerCallback(void* user_data,
}

Local<Value> result;
if (!retval.ToLocal(&result) || result->IsUndefined() || result->IsNull()) {
// Missing return value
if (!retval.ToLocal(&result) || result->IsUndefined() || result->IsNull() ||
!result->IsInt32()) {
Local<Value> err = Exception::TypeError(
String::NewFromUtf8(
isolate,
Expand All @@ -1978,15 +1978,6 @@ int DatabaseSync::AuthorizerCallback(void* user_data,
return SQLITE_DENY;
}

if (!result->IsInt32()) {
Local<Value> err = Exception::TypeError(
String::NewFromUtf8(
isolate, "Authorizer callback return value must be an integer")
.ToLocalChecked());
db->StoreAuthorizerError(err);
return SQLITE_DENY;
}

int32_t int_result = result.As<Int32>()->Value();
if (int_result != SQLITE_OK && int_result != SQLITE_DENY &&
int_result != SQLITE_IGNORE) {
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-sqlite-authz.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ suite('DatabaseSync.prototype.setAuthorizer()', () => {
assert.throws(() => {
db.exec('SELECT 1');
}, {
message: 'Authorizer callback return value must be an integer'
message: 'Authorizer callback must return an integer authorization code'
});
});

Expand Down
Loading