Skip to content

Remove SSL Certifcate auth reliance on internal password - #2674

Merged
carsongee merged 2 commits into
openedx:masterfrom
carsongee:cg/ssl_remove_password
Feb 21, 2014
Merged

Remove SSL Certifcate auth reliance on internal password#2674
carsongee merged 2 commits into
openedx:masterfrom
carsongee:cg/ssl_remove_password

Conversation

@carsongee

Copy link
Copy Markdown
Contributor

This was suggested long ago in #1182 and earlier in #653 by @brianhw and @jbau so that there won't be a mix up between the user's internal password and this password, but still allows the user to have and maintain their own password inside the standard model auth. Heads up to @sarina @ichuang @pdpinch

@sarina

sarina commented Feb 21, 2014

Copy link
Copy Markdown
Contributor

👍 by me. @jbau says he'll take a look -- feel free to merge once he gives it his blessing.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

typo: should be 'django.contrib.auth.backends.ModelBackend'

@carsongee

Copy link
Copy Markdown
Contributor Author

Pedantic is fine as long as its quick and easy. Everything is fixed up

@jbau

jbau commented Feb 21, 2014

Copy link
Copy Markdown

👍

carsongee added a commit that referenced this pull request Feb 21, 2014
Remove SSL Certifcate auth reliance on internal password
@carsongee
carsongee merged commit 2da1e4e into openedx:master Feb 21, 2014
jenkins-ks pushed a commit to nttks/edx-platform that referenced this pull request Jun 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

open-source-contribution PR author is not from Axim or 2U

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants