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

Fix insdcIngestUserPassword to fix ingest pipeline erros #1808

Merged
merged 1 commit into from
May 8, 2024

Conversation

anna-parker
Copy link
Contributor

@anna-parker anna-parker commented May 8, 2024

Fixes

Currently, I am not able to run the ingest pipeline locally as snakemake gives me an access denied warning:

"/Users/aparker/Documents/Github/loculus/ingest/scripts/submit_to_loculus.py", line 51, in get_jwt
    response.raise_for_status()
  File "/Users/aparker/micromamba/envs/loculus-ingest/lib/python3.12/site-packages/requests/models.py", line 1021, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://authentication-main.loculus.org/realms/loculus/protocol/openid-connect/token

Changing the insdcIngestUserPassword to "insdc_ingest_user" fixes this issue.
Looking back this file was only added in #1734 - previously the password was set here:
image
And so it seems like it was just incorrectly copied over.

Summary

Correct the insdcIngestUserPassword

@anna-parker anna-parker marked this pull request as ready for review May 8, 2024 08:12
Copy link
Member

@theosanderson theosanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, and apologies!

@anna-parker anna-parker merged commit 893d2c7 into main May 8, 2024
8 checks passed
@anna-parker anna-parker deleted the fix/insdc_error branch May 8, 2024 08:31
@anna-parker
Copy link
Contributor Author

Thank you, and apologies!

No problem!

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

Successfully merging this pull request may close these issues.

3 participants