Closed collinpreston closed 1 year ago
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
Our GitHub checks need improvements? Share your feedbacks!
Secret key warning appears to be a false positive triggered by formatting the settings.py file. No value has been added or changed on this branch.
Merging #5264 (f454cbe) into master (22fde05) will increase coverage by
0.00%
. The diff coverage is90.90%
.
@@ Coverage Diff @@
## master #5264 +/- ##
=======================================
Coverage 93.68% 93.68%
=======================================
Files 501 501
Lines 23273 23275 +2
Branches 965 965
=======================================
+ Hits 21804 21806 +2
Misses 1362 1362
Partials 107 107
Impacted Files | Coverage Δ | |
---|---|---|
certificates/views.py | 94.69% <ø> (ø) |
|
courses/models.py | 97.52% <ø> (ø) |
|
financialaid/views.py | 97.20% <ø> (ø) |
|
ui/views.py | 92.30% <ø> (ø) |
|
backends/base.py | 82.35% <66.66%> (ø) |
|
backends/constants.py | 100.00% <100.00%> (ø) |
|
backends/edxorg.py | 100.00% <100.00%> (ø) |
|
dashboard/api.py | 96.43% <100.00%> (ø) |
|
ecommerce/views.py | 92.06% <100.00%> (ø) |
|
micromasters/settings.py | 89.80% <100.00%> (+0.04%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
@rachellougee, yes before we release this we should create the EDXORG_CALLBACK_URL
environment variable and set it equal to the same value used for EDXORG_BASE_URL
in RC and Production.
Related DevOps ticket for adding the environment variable: https://github.com/mitodl/ol-infrastructure/issues/1247.
Pre-Flight checklist
app.json
What's this PR do?
This allows developers to more easily get OAuth SSO working between Micromasters and edX (Devstack) when running both locally. As part of this change, I am using
EDXORG_BASE_URL
as the outbound base URL for edX andEDXORG_CALLBACK_URL
for the inbound base URL for edX which is why you will see a lot of the original instances ofEDXORG_BASE_URL
have now been changed toEDXORG_CALLBACK_URL
.How should this be manually tested?
Follow the updated instructions for setting up Micromasters. If you already have Micromasters running locally, then replace the following with the values shown in your
.env
file:Verify the functionality of this PR be ensuring that you can create a new Micromasters account via SSO with edX as well as login to an existing Micromasters account via SSO with edX.