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.

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

Jump to bottom

🐛 Bug Report — Runtime APIs - fts5 allowed but not fts5vocab #1540

Closed
rozenmd opened this issue Jan 16, 2024 · 2 comments
Closed

🐛 Bug Report — Runtime APIs - fts5 allowed but not fts5vocab #1540

rozenmd opened this issue Jan 16, 2024 · 2 comments
Labels
feature request Request for Workers team to add a feature sqlite

Comments

@rozenmd
Copy link
Contributor

rozenmd commented Jan 16, 2024

The fts5vocab virtual table module allows users to extract information from an FTS5 full-text index directly. The fts5vocab module is a part of FTS5 - it is available whenever FTS5 is.

from https://www.sqlite.org/fts5.html

attempting to use fts5vocab in a query throws an error:

> CREATE VIRTUAL TABLE ft1_v USING fts5vocab('ft1', 'Order');
not authorized

Issue originally raised here: cloudflare/workers-sdk#4543
@jasnell jasnell added feature request Request for Workers team to add a feature sqlite labels Feb 2, 2024
mhart added a commit that referenced this issue Jun 12, 2024
@mhart
Allow fts5vocab module (#1540)
360bbf6 
This module was inadvertently missed in the original allow list – it's a standard part of of FTS5. As per the docs: "[fts5vocab] is available whenever FTS5 is"

https://www.sqlite.org/fts5.html#the_fts5vocab_virtual_table_module
mhart added a commit that referenced this issue Jun 12, 2024
@mhart
Allow fts5vocab module (#1540)
3cdcd61 
This module was inadvertently missed in the original allow list – it's a standard part of of FTS5. As per the docs: "[fts5vocab] is available whenever FTS5 is"

https://www.sqlite.org/fts5.html#the_fts5vocab_virtual_table_module
mhart added a commit that referenced this issue Jun 12, 2024
@mhart
Merge pull request #2259 from cloudflare/mhart/1540/allow-list-fts5vocab
5f54478 
Allow fts5vocab module (#1540)
@mhart
Copy link
Collaborator

mhart commented Jun 12, 2024

Fixed in #2259

@mhart mhart closed this as completed Jun 12, 2024
@naourass
Copy link

naourass commented Sep 28, 2024 •
edited
Loading

Update: just updated wrangler to the latest version and the migration went through without any error :) So just update your wrangler if anyone runs into this same issue.

Was this fix published? I'm trying to create an fts5vocab table using a migration file but getting the not authorized error even when trying to apply the migration locally:

CREATE VIRTUAL TABLE articles_fts_vocab USING fts5vocab(articles_fts, 'row');

Error after wrangler migration command:

❌ Migration xxxxxxx.sql failed with the following errors:
✘ [ERROR] not authorized

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request Request for Workers team to add a feature sqlite
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mhart @jasnell @rozenmd @naourass