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

Access key authentication #218

Closed
chaoran-chen opened this issue Apr 5, 2023 · 0 comments · Fixed by #238 or #778
Closed

Access key authentication #218

chaoran-chen opened this issue Apr 5, 2023 · 0 comments · Fixed by #238 or #778
Assignees
@fengelniederhammer
Labels
LAPIS Tasks/PR related to LAPIS
Milestone
COVID-LAPIS

Comments

@chaoran-chen
Copy link
Member

chaoran-chen commented Apr 5, 2023 •
edited
Loading

For non-open instances such as the GISAID one, we only allow access if an access key is provided.

  • It should be possible to specify the data openness level in the config. There should be two openness levels: open and gisaid.
  • If the openness level is gisaid, the server should only allow access with a valid access key. In the current LAPIS, the access keys are stored in the PostgreSQL database. As we probably won't have an external relational database in LAPIS v2, we can also just keep them in a file.
@chaoran-chen chaoran-chen added the LAPIS Tasks/PR related to LAPIS label Apr 5, 2023
@chaoran-chen chaoran-chen added this to the COVID-LAPIS milestone Apr 5, 2023
@fengelniederhammer fengelniederhammer self-assigned this Apr 13, 2023
fengelniederhammer added a commit that referenced this issue Apr 13, 2023
@fengelniederhammer
feat: base implementation to provide the openness level in the config
62fd1f7 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 17, 2023
@fengelniederhammer
feat: get access key from request and read valid access keys from file
79d4cb0 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 19, 2023
@fengelniederhammer
feat: base implementation to provide the openness level in the config
b86cc1d 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 19, 2023
@fengelniederhammer
feat: get access key from request and read valid access keys from file
9cb7953 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 24, 2023
@fengelniederhammer
feat: base implementation to provide the openness level in the config
e39738a 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 24, 2023
@fengelniederhammer
feat: get access key from request and read valid access keys from file
5bd08aa 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 25, 2023
@fengelniederhammer
feat: get access key from request and read valid access keys from file
f1a1a3c 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 25, 2023
@fengelniederhammer
feat: document access key in swagger docs
cc069e0 
issue: #218
@fengelniederhammer fengelniederhammer mentioned this issue Apr 25, 2023
Merged
@fengelniederhammer fengelniederhammer linked a pull request Apr 25, 2023 that will close this issue
Access key authentication #238
Merged
fengelniederhammer added a commit that referenced this issue Apr 27, 2023
@fengelniederhammer
feat: base implementation to provide the openness level in the config
9919dba 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 27, 2023
@fengelniederhammer
feat: get access key from request and read valid access keys from file
9d334bd 
issue: #218
fengelniederhammer added a commit that referenced this issue Apr 27, 2023
@fengelniederhammer
feat: document access key in swagger docs
1e1fe00 
issue: #218
fengelniederhammer added a commit that referenced this issue May 2, 2023
@fengelniederhammer
feat: base implementation to provide the openness level in the config
aff116c 
issue: #218
fengelniederhammer added a commit that referenced this issue May 2, 2023
@fengelniederhammer
feat: get access key from request and read valid access keys from file
4e07a6b 
issue: #218
fengelniederhammer added a commit that referenced this issue May 2, 2023
@fengelniederhammer
feat: document access key in swagger docs
0cd3070 
issue: #218
fengelniederhammer added a commit that referenced this issue May 2, 2023
@fengelniederhammer
refactor: don't mention GISAID in openness level to keep it general
31f95d5 
issue: #218
This was referenced May 6, 2024
Closed
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fengelniederhammer fengelniederhammer

Labels
LAPIS Tasks/PR related to LAPIS
Projects
None yet
Milestone
COVID-LAPIS
Development

Successfully merging a pull request may close this issue.

Access key authentication GenSpectrum/LAPIS
chore: release main GenSpectrum/LAPIS
2 participants
@chaoran-chen @fengelniederhammer