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

Rename NB_API_ALLOWED_ORIGINS to NB_NAPI_ALLOWED_ORIGINS #410

Open
alyssadai opened this issue Feb 13, 2025 · 0 comments
Open

Rename NB_API_ALLOWED_ORIGINS to NB_NAPI_ALLOWED_ORIGINS #410

alyssadai opened this issue Feb 13, 2025 · 0 comments

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Feb 13, 2025

NB_NAPI_ALLOWED_ORIGINS is the name of the configuration variable in our deployment recipe, which maps to NB_API_ALLOWED_ORIGINS for the API when it is deployed as a Docker container.

To avoid confusion for ourselves and users, we should use the same name for both the environment variable as expected by the API itself, and the environment variable we expect the user to change in their recipe.

Note: neurobagel/recipes will also need to be updated

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

No branches or pull requests

1 participant