Merge lp:~danilo/linaro-license-protection/openid-enablement into lp:~linaro-automation/linaro-license-protection/trunk

Proposed by Данило Шеган
Status: Merged
Approved by: Stevan Radaković
Approved revision: 104
Merged at revision: 104
Proposed branch: lp:~danilo/linaro-license-protection/openid-enablement
Merge into: lp:~linaro-automation/linaro-license-protection/trunk
Diff against target: 70 lines (+15/-13)
2 files modified
settings.py (+10/-8)
urls.py (+5/-5)
To merge this branch: bzr merge lp:~danilo/linaro-license-protection/openid-enablement
Reviewer Review Type Date Requested Status
Stevan Radaković Approve
Review via email: mp+120080@code.launchpad.net

Description of the change

Change OpenID configuration so it can easily coexist with apache openid
protection.

To post a comment you must log in.
Revision history for this message
Stevan Radaković (stevanr) wrote :

Good work, thanks.
Approve +1.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'settings.py'
--- settings.py 2012-08-01 13:46:32 +0000
+++ settings.py 2012-08-17 08:39:19 +0000
@@ -96,7 +96,9 @@
9696
97ROOT_URLCONF = ROOT_DIR + '.urls'97ROOT_URLCONF = ROOT_DIR + '.urls'
9898
99TEMPLATE_DIRS = (os.path.join(PROJECT_ROOT, "templates" ),)99TEMPLATE_DIRS = (
100 os.path.join(PROJECT_ROOT, "templates" ),
101 )
100102
101INSTALLED_APPS = (103INSTALLED_APPS = (
102 'django.contrib.auth',104 'django.contrib.auth',
@@ -105,11 +107,8 @@
105 'django.contrib.sites',107 'django.contrib.sites',
106 'django.contrib.messages',108 'django.contrib.messages',
107 'django.contrib.staticfiles',109 'django.contrib.staticfiles',
108 # Uncomment the next line to enable the admin:
109 'django.contrib.admin',110 'django.contrib.admin',
110 'django_openid_auth',111 'django_openid_auth',
111 # Uncomment the next line to enable admin documentation:
112 # 'django.contrib.admindocs',
113 ROOT_DIR + '.license_protected_downloads',112 ROOT_DIR + '.license_protected_downloads',
114)113)
115114
@@ -118,13 +117,16 @@
118 'django.contrib.auth.backends.ModelBackend',117 'django.contrib.auth.backends.ModelBackend',
119)118)
120119
121LOGIN_URL = '/openid/login/'120LOGIN_URL = '/linaro-openid/login/'
121LOGIN_REDIRECT_URL = '/'
122122
123OPENID_CREATE_USERS = True123OPENID_CREATE_USERS = True
124OPENID_SSO_SERVER_URL = 'https://login.launchpad.net/'124OPENID_SSO_SERVER_URL = 'https://login.launchpad.net/'
125OPENID_LAUNCHPAD_TEAMS_MAPPING = {125OPENID_UPDATE_DETAILS_FROM_SREG = True
126 'linaro': 'linaro',126OPENID_LAUNCHPAD_TEAMS_MAPPING_AUTO = True
127}127OPENID_FOLLOW_RENAMES = True
128OPENID_USE_AS_ADMIN_LOGIN = True
129OPENID_USE_EMAIL_FOR_USERNAME = True
128130
129# A sample logging configuration. The only tangible logging131# A sample logging configuration. The only tangible logging
130# performed by this configuration is to send an email to132# performed by this configuration is to send an email to
131133
=== modified file 'urls.py'
--- urls.py 2012-07-11 13:08:55 +0000
+++ urls.py 2012-08-17 08:39:19 +0000
@@ -6,13 +6,13 @@
66
77
8urlpatterns = patterns('',8urlpatterns = patterns('',
9 # Uncomment the admin/doc line below to enable admin documentation:
10 url(r'^admin/doc/', include('django.contrib.admindocs.urls')),
11
12 # Uncomment the next line to enable the admin:
13 url(r'^admin/', include(admin.site.urls)),9 url(r'^admin/', include(admin.site.urls)),
1410
15 url(r'^openid/', include('django_openid_auth.urls')),11 # Use "linaro-openid" to allow peaceful coexistence of both
12 # python-apache-openid and django-openid authentication on the
13 # same server. When we get rid of apache openid protection,
14 # we can go back to using just "openid" here.
15 url(r'^linaro-openid/', include('django_openid_auth.urls')),
16 url(r'^logout/$', 'django.contrib.auth.views.logout'),16 url(r'^logout/$', 'django.contrib.auth.views.logout'),
1717
18 # The license page...18 # The license page...

Subscribers

People subscribed via source and target branches