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

Explcitly set OAUTHLIB_INSECURE_TRANSPORT env var when present in .env #928

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

ml-evs
Copy link
Member

@ml-evs ml-evs commented Oct 14, 2024

Previously, the docker-based dev server implicitly called load_dotenv to load any environment variables into global state, as part of flask run. Since the dev containers have now been removed, it is useful to be able to control a few select env vars in this way through the gunicorn-based container.

This PR hardcodes the setting of OAUTHLIB_INSECURE_TRANSPORT as an env var, when specified in the .env file. This allows OAuth login to be used locally without https.

Copy link

cypress bot commented Oct 14, 2024

datalab    Run #2553

Run Properties:  status check passed Passed #2553  •  git commit 587ea439d4 ℹ️: Merge d2242825ec76ef9a10c51f49166c9dc47a4700b0 into 937d9dfa0609ef54dd3bb2e0be73...
Project datalab
Run status status check passed Passed #2553
Run duration 06m 18s
Commit git commit 587ea439d4 ℹ️: Merge d2242825ec76ef9a10c51f49166c9dc47a4700b0 into 937d9dfa0609ef54dd3bb2e0be73...
Committer Matthew Evans
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 396

@ml-evs ml-evs force-pushed the ml-evs/tweak-dev-server branch from 45cca10 to d224282 Compare October 14, 2024 17:21
@ml-evs ml-evs merged commit a61e64f into main Oct 14, 2024
15 checks passed
@ml-evs ml-evs deleted the ml-evs/tweak-dev-server branch October 14, 2024 17:34
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.

1 participant