update: no logout, simple refactor

djangoldp_account/endpoints/rp_login.py

@@ -14,7 +14,7 @@ from oic.utils.authn.client import CLIENT_AUTHN_METHOD
 
 class RPLoginEndpoint(object):
     """
-    RL login endpoint
+    RP login endpoint
     This is *not* the OP login endpoint (OP part is managed by django_oidc)
     """
     client_class = Client
@@ -51,7 +51,7 @@ class RPLoginEndpoint(object):
 
     def op_login_url(self):
         subject = self.params['subject']
-        redirect_uris = [settings.SITE_URL + reverse('ldp_login_callback'), settings.SITE_URL]
+        redirect_uris = [settings.SITE_URL + reverse('oidc_login_callback'), settings.SITE_URL]
         try:
             issuer = self.client.discover(subject)
             provider_info = self.client.provider_config(issuer)

djangoldp_account/errors.py

 class LDPLoginError(Exception):
-    """
-    """
-
     _errors = {
         'invalid_request': "The request provider parameter must contains an url or an email",
         'invalid_state': "Invalid state",

djangoldp_account/templates/registration/login.html

@@ -38,7 +38,7 @@
 
     <p> or login using another authentification provider</p>
 
-<form method="post" action="{% url 'ldp_login' %}">
+<form method="post" action="{% url 'oidc_login' %}">
     {% csrf_token %}
     <table>
         <tr>

djangoldp_account/urls.py

@@ -5,8 +5,6 @@ from django.conf import settings
 from django.conf.urls import url, include
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import Group
-from django.urls import reverse_lazy
-from django.views.generic import RedirectView
 
 from djangoldp.permissions import AnonymousReadOnly
 from djangoldp.views import LDPViewSet
@@ -33,8 +31,8 @@ urlpatterns = [
     url(r'^accounts/', include('django.contrib.auth.urls')),
     url(r'^accounts/', LDPViewSet.urls(model=Account, permission_classes=[AnonymousReadOnly])),
     url(r'^chat-profile/?', LDPViewSet.urls(model=ChatProfile, permission_classes=[AnonymousReadOnly])),
-    url(r'^ldp/login/callback/?$', RPLoginCallBackView.as_view(), name='ldp_login_callback'),
-    url(r'^ldp/login/?$', RPLoginView.as_view(), name='ldp_login'),
+    url(r'^oidc/login/callback/?$', RPLoginCallBackView.as_view(), name='oidc_login_callback'),
+    url(r'^oidc/login/?$', RPLoginView.as_view(), name='oidc_login'),
     url(r'^openid/userinfo/?$', userinfocustom),
     url(r'^openid/', include('oidc_provider.urls', namespace='oidc_provider'))
 ]

djangoldp_account/views.py

@@ -62,4 +62,3 @@ class RPLoginCallBackView(View):
 
     def post(self, request, *args, **kwargs):
         return self.on_request(request)
-