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

Expose auth variables to n-API #66

Closed
alyssadai opened this issue Jul 23, 2024 · 1 comment · Fixed by #67
Closed

Expose auth variables to n-API #66

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

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Jul 23, 2024 •
edited
Loading

  • n-API and local query tool currently are missing authentication environment variables in the Compose config
  • Also, default value for NB_ENABLE_AUTH should be false, since this is still an experimental feature (and to match the default in the query tool)
@alyssadai alyssadai moved this to Backlog in Neurobagel Jul 23, 2024
@alyssadai alyssadai moved this from Backlog to Implement - Active in Neurobagel Jul 25, 2024
@alyssadai alyssadai self-assigned this Jul 25, 2024
@alyssadai alyssadai mentioned this issue Jul 25, 2024
Merged
6 tasks
@alyssadai alyssadai moved this from Implement - Active to Implement - Done in Neurobagel Jul 25, 2024
@rmanaem rmanaem moved this from Implement - Done to Review - Active in Neurobagel Jul 26, 2024
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Jul 29, 2024
@neurobagel-bot Neurobagel Bot
Copy link
Contributor

neurobagel-bot bot commented Aug 7, 2024

🚀 Issue was released in v0.0.1 🚀

@neurobagel-bot neurobagel-bot bot added the released This issue/pull request has been released. label Aug 7, 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
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] add auth vars to n-API & local query tool and disable by default neurobagel/recipes
1 participant
@alyssadai