Skip to content

Commit

Permalink
Merge pull request #59 from bento-platform/chore/add-validate-ssl-inject
Browse files Browse the repository at this point in the history
chore: add validate_ssl injectable config
  • Loading branch information
davidlougheed authored Nov 20, 2023
2 parents 38474b9 + 3e60ed2 commit df5ceb9
Show file tree
Hide file tree
Showing 3 changed files with 189 additions and 197 deletions.
6 changes: 5 additions & 1 deletion bento_wes/runs.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,12 @@ def _check_single_run_permission_and_mark(run_req: RunRequest, permission: str)
) if authz_enabled() else True


def _config_for_run(run_dir: str) -> dict[str, str | None]:
def _config_for_run(run_dir: str) -> dict[str, str | bool | None]:
return {
# In production, workflows should validate SSL (i.e., omit the curl -k flag).
# In development, SSL certificates are usually self-signed, so they will not validate.
"validate_ssl": current_app.config["BENTO_VALIDATE_SSL"],

# In export/analysis mode, as we rely on services located in different containers
# there is a need to have designated folders on shared volumes between
# WES and the other services, to write files to.
Expand Down
Loading

0 comments on commit df5ceb9

Please sign in to comment.