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

"Read-only" API key idea #196

Open
justinclift opened this issue Apr 30, 2023 · 6 comments
Open

"Read-only" API key idea #196

justinclift opened this issue Apr 30, 2023 · 6 comments

Comments

@justinclift
Copy link
Member

@chrisjlocke mentioned that having "read-only" api keys would be useful.

There's an outstanding task to do fairly fine grained API keys, but that's going to take some time.

It would be pretty easy to implement read-only API keys (not fine grained) in the meantime.

eg "if the caller uses this API key, they can only query stuff"

@justinclift
Copy link
Member Author

justinclift commented Apr 30, 2023

Hmmm, thinking that through more, we'd probably still need to ensure that "read only" API keys are limited to a single database.

The use case is things like:

I want to embed an API key in my serverless application, which reads [stuff] from database XYZ.

For that kind of scenario, the API key shouldn't have access to do things like "get a list of all databases, including private ones", "read the data of (other) private databases", etc.

So, limiting them to a single database seems like the right approach.

@mshiner
Copy link

mshiner commented Aug 23, 2023

Justin

"I want to embed an API key in my serverless application, which reads [stuff] from database XYZ." is exactly my use case so.... yes please.

M

@captn3m0
Copy link

I tried to work around this by sharing a database with a second user with "read-only-access", and then using an API key from the second user. That didn't work.

@justinclift
Copy link
Member Author

Ahhh. Yeah, this is still on the ToDo list (development has picked up again recently).

@MKleusberg This is another potential thing you could look at. It has a need for React stuff too, if that's good. 😄

@MKleusberg
Copy link
Member

This has finally been implemented in a59f8dc now.

@justinclift
Copy link
Member Author

Cool. I'm about to hit the sack here, but tomorrow I'll put time into reviewing and testing the change then potentially deploy it to production. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants