-
Notifications
You must be signed in to change notification settings - Fork 34
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
Ensure that the token list is stored on the server so that the same image works for both studies and programs #790
Comments
e-mission-server supports multiple different authentication methods.
One of them is This is implemented at |
Options are:
|
Cleanest solution appears to be the database.
The actual task for @aGuttman is:
|
|
Wed: write a script to load a token list into a table (aka collection in mongodb) into the database. make sure to access it via wrapper code in emission/storage so that if we do want to support a different database (e.g. couchdb), it should be easy to find all locations. |
This issue worked on in PR e-mission/e-mission-server#885 |
wrt "read from the dynamic config on the public github instead", the server does not currently read from the public github, which is why it is not dynamically configured per the title of this issue. However, the viz_scripts do read from the dynamic config in python to determine which graphs to generate and you should be able to reuse that code. https://github.com/e-mission/em-public-dashboard/blob/main/viz_scripts/bin/generate_plots.py#L26 |
At least initially, it would be good if we would be able to support non-dynamic config as well, for those community members that do not want to have dynamically configurable instances. So I would suggest creating a new auth method called "dynamic", read the config in there, and then choose skip or token_list from there depending on the config. Another aspect to consider during the design is that you don't want to read the config for every auth because that would be too much of an overhead; we should read the config once at startup or once a day or something and then use it forever. |
My approach is to resolve i.e. if This avoids the problem of reading from the dynamic conf every time we want to verify a token, as adding a |
server code is checked in |
docker-compose changes are complete and solution has been deployed. |
concretely, how to configure a list of tokens on the production server and to potentially have other program vs. study differences.
Dynamic configs are stored at https://github.com/e-mission/nrel-openpath-deploy-configs
Please ask @zackAemmer or @sebastianbarry if you have any questions about dynamic configs since they have worked with them recently.
The text was updated successfully, but these errors were encountered: