Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

[bug 1333743] update documentation for new auth #1583

Closed
wants to merge 3 commits into from
Closed

[bug 1333743] update documentation for new auth #1583

wants to merge 3 commits into from

Conversation

Mte90
Copy link
Contributor

@Mte90 Mte90 commented Jan 25, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.01% when pulling 14d1dff on Mte90:master into 35ea5cc on mozilla:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.01% when pulling 14d1dff on Mte90:master into 35ea5cc on mozilla:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.01% when pulling 14d1dff on Mte90:master into 35ea5cc on mozilla:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 92.91% when pulling 194c90e on Mte90:master into 35ea5cc on mozilla:master.

@Mte90
Copy link
Contributor Author

Mte90 commented Feb 14, 2017

Any hope for this one?

@@ -42,6 +42,14 @@ ADMINS = (
# ('Your Name', 'your_email@domain.com'),
)

OIDC_OP_AUTHORIZATION_ENDPOINT = ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of replacing these entries with empty strings please add the follow to local.py dist files to force using the ModelBackend for authentication.

AUTHENTICATION_BACKENDS = ( 'django.contrib.auth.backends.ModelBackend', )


.. note::

In case this command doesn't work, you can run ``./scripts/su.sh`` inside the container. In order to get shell access please run ``docker-compose run web /bin/bash``.
To access in the site: http://127.0.0.1:8000/admin, the button signin cannot work.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please replace this with:
To login to mozillians.org for dev purposes please visit http://127.0.0.1:8000/admin and login with the credentials issues in the previous step. The signin button doesn't work locally.

@@ -36,6 +36,14 @@ ADMINS = (
# ('Your Name', 'your_email@domain.com'),
)

OIDC_OP_AUTHORIZATION_ENDPOINT = ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

@@ -90,3 +98,6 @@ CSP_STYLE_SRC = base.CSP_STYLE_SRC + (
BASKET_URL = 'http://127.0.0.1'
BASKET_API_KEY = 'basket_api_key'
BASKET_MANAGERS = None # or list of email addresses.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a comment to mention that these are dev keys that always return True.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -50,6 +50,14 @@ ADMINS = (
# ('Your Name', 'your_email@domain.com'),
)

OIDC_OP_AUTHORIZATION_ENDPOINT = ''
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

@@ -116,3 +124,6 @@ CSP_STYLE_SRC = base.CSP_STYLE_SRC + (
'http://*.mozilla.org',
'http://*.mapbox.com',
)

NORECAPTCHA_SITE_KEY = '6LeIxAcTAAAAAJcZVRqyHh71UMIEGNQ_MXjiZKhI'
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 92.953% when pulling 799f8b9 on Mte90:master into 35ea5cc on mozilla:master.

@johngian
Copy link
Contributor

Closing this one after #1623

@johngian johngian closed this Feb 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants