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 issue in WDQS container #475

Merged
merged 1 commit into from
Jan 25, 2024
Merged

Fix issue in WDQS container #475

merged 1 commit into from
Jan 25, 2024

Conversation

physikerwelt
Copy link
Member

MaRDI Pull Request

For some reason the file in the WDQS container is
called whitelist.txt while the service defaults to a file named allowlist.txt

Closes #473

Instructions for PR review:

  • Conceptual Review (Logic etc...)
  • Code Review (Review your implementation)
  • Checkout (Test changes locally) Tested by setting the environment variable temporary on in production

Checklist for this PR:

For some reason the file in the WDQS container is
called whitelist.txt while the service defaults to a file
named allowlist.txt

Closes #473
@physikerwelt physikerwelt merged commit 145adbd into main Jan 25, 2024
3 checks passed
@physikerwelt physikerwelt deleted the WDQS_ALLOWLIST branch January 25, 2024 19:44
@JeroenDeDauw
Copy link
Contributor

called whitelist.txt while the service defaults to a file named allowlist.txt

image

@physikerwelt
Copy link
Member Author

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.

Re-Enable federated queries
3 participants