tursodatabase / tursodatabase/libsql
Improve authentication failure error
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- C
- Stars
- 17.2k
- Forks
- 531
- Avg merge
- 1h 12m
- Merged PRs (30d)
- 1
Description
Currently, if authentication fails, we send return confusing gRPC error message. Let's add a proper AuthenticationFailed error to make the error case more obvious.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No file, test, or entry point is named. Start by locating the authentication failure path and the gRPC error construction, then check nearby error tests or callers; done means authentication failures return the distinct AuthenticationFailed error with an unambiguous message.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c, grpc
- Domain
- authentication, backend-api-design
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100