Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

basic_auth should be required for writer API #34

Open
piyusql opened this issue May 31, 2023 · 0 comments
Open

basic_auth should be required for writer API #34

piyusql opened this issue May 31, 2023 · 0 comments
Assignees
@preyunk
Labels
enhancement New feature or request
Milestone
v0.2.4

Comments

@piyusql
Copy link
Contributor

piyusql commented May 31, 2023

Although the read api can serve without authentication by default.
But strongly we should use authentication for any updates in our database.

https://github.com/vmware/bridgeql/blob/cb3055ac88f352d2531d1e0cd4036e96a4e0d805/bridgeql/django/settings.py#L16

we can use bridgeql.django.auth.basic_auth as default writer option.
Please update README accordingly.
@piyusql piyusql added the enhancement New feature or request label May 31, 2023
@piyusql piyusql added this to the v0.2.4 milestone May 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@preyunk preyunk

Labels
enhancement New feature or request
Projects
None yet
Milestone
v0.2.4
Development

No branches or pull requests
2 participants
@piyusql @preyunk