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.

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

Users are struggling when creating tenant token #2786

Open
curquiza opened this issue Apr 15, 2024 · 4 comments · May be fixed by #2832
Open

Users are struggling when creating tenant token #2786

curquiza opened this issue Apr 15, 2024 · 4 comments · May be fixed by #2832

Comments

@curquiza
Copy link
Member

curquiza commented Apr 15, 2024

A user struggled to understand why the search requests were failing with their tenant token

meilisearch/meilisearch#4567 (comment)

They wanted to access the full index but without specifying any filter, so they needed to add indexName: {} which was not intuitive. Maybe we could detail this?

@guimachiavelli, to be honest, I did not check in the documentation if this explanation is already available, so feel free to close this issue if not useful

However I think it's still useful to work on tenant token experience. To let you know, and maybe we could do something on the product side as well: users often struggle with tenant token usage because they DON'T know why the request is forbidden (i.e. the tenant token is invalid). The message is really vague and does not indicate if the action or the index is forbidden. So maybe we could add some warning of basic errors to check have having issues with tenant token?

@curquiza curquiza changed the title Strangle when creating tenant token Struggle when creating tenant token Apr 15, 2024
@curquiza curquiza changed the title Struggle when creating tenant token Users are struggling when creating tenant token Apr 15, 2024
@guimachiavelli
Copy link
Member

Reworking the tenant tokens article is in this quarter's roadmap. I'll make sure to address the indexName: {} bit when I get to that.

@guimachiavelli
Copy link
Member

As noted by the support team, whether the API key uuid is necessary is not communicated very clearly in the tenant token guide. Fixing that should also be part of this rework.

Reference link (private slack chat): https://meilisearch.slack.com/archives/CD7AJLL01/p1715863916526779

@irevoire
Copy link
Member

The message is really vague and does not indicate if the action or the index is forbidden. So maybe we could add some warning of basic errors to check have having issues with tenant token?

I also believe it’s something we should fix from the engine directly

@curquiza
Copy link
Member Author

I agree, pinging @gmourier and @macraig for the product part, since we have product usage frustration here

@guimachiavelli guimachiavelli linked a pull request May 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants