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

Add authentication to /query route #322

Closed
4 tasks done
alyssadai opened this issue Jul 12, 2024 · 1 comment · Fixed by #323
Closed
4 tasks done

Add authentication to /query route #322

alyssadai opened this issue Jul 12, 2024 · 1 comment · Fixed by #323
Assignees
@alyssadai
Labels
released This issue/pull request has been released.

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Jul 12, 2024 •
edited

  • add dependency of /query route on OAuth2 scheme
  • verify ID token from header using google-auth
  • read in client ID from env var, and error if unset
  • have option to turn off authentication
@alyssadai alyssadai changed the title Add authentication to /query Add authentication to /query route Jul 12, 2024
@rmanaem rmanaem mentioned this issue Jul 15, 2024
Closed
4 tasks
@alyssadai alyssadai self-assigned this Jul 15, 2024
@alyssadai alyssadai mentioned this issue Jul 17, 2024
Merged
7 tasks
@neurobagel-bot
Copy link

neurobagel-bot bot commented Aug 2, 2024

🚀 Issue was released in v0.3.0 🚀

@neurobagel-bot neurobagel-bot bot added the released This issue/pull request has been released. label Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alyssadai alyssadai

Labels
released This issue/pull request has been released.
Projects
Neurobagel
Status: Review - Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] Add authentication to /query route neurobagel/api
1 participant
@alyssadai