Bump django-allauth from 0.51.0 to 65.13.0 in /requirements (mozilla#3867)

In addition to resolving all backwards incompatibilities from 0.51.0 to 65.13.0, this PR also drops the redirect on Email consent page, because it was buggy.

Author: dependabot[bot]

pontoon/allauth_urls.py

@@ -4,10 +4,9 @@
 views and don't allow user to tamper with the state of an account.
 """
 
-import importlib
-
 from allauth.account import views as account_views
-from allauth.socialaccount import providers, views as socialaccount_views
+from allauth.socialaccount import views as socialaccount_views
+from allauth.urls import build_provider_urlpatterns
 
 from django.conf import settings
 from django.contrib.auth import views
@@ -25,26 +24,19 @@
     ]
 else:
     urlpatterns = [
+        path("signup/", account_views.signup, name="account_signup"),
         path("login/", account_views.login, name="account_login"),
         path("logout/", account_views.logout, name="account_logout"),
         path("inactive/", account_views.account_inactive, name="account_inactive"),
         path(
-            "social/login/cancelled/",
+            "/login/cancelled/",
             socialaccount_views.login_cancelled,
             name="socialaccount_login_cancelled",
         ),
         path(
-            "social/login/error/",
+            "/login/error/",
             socialaccount_views.login_error,
             name="socialaccount_login_error",
         ),
     ]
-
-for provider in providers.registry.get_list():
-    try:
-        prov_mod = importlib.import_module(provider.get_package() + ".urls")
-    except ImportError:
-        continue
-    prov_urlpatterns = getattr(prov_mod, "urlpatterns", None)
-    if prov_urlpatterns:
-        urlpatterns += prov_urlpatterns
+    urlpatterns += build_provider_urlpatterns()

…tes/django/account/account_inactive.html …/templates/account/account_inactive.htmlpontoon/base/templates/django/account/account_inactive.html renamed to pontoon/base/templates/account/account_inactive.html pontoon/base/templates/django/account/account_inactive.html renamed to pontoon/base/templates/account/account_inactive.html

@@ -1,4 +1,4 @@
-{% extends "base.html" %}
+{% extends "allauth/layouts/base.html" %}
 {% load static %}
 
 {% block title %}Account Disabled{% endblock %}

pontoon/base/templates/django/base.html …base/templates/allauth/layouts/base.htmlpontoon/base/templates/django/base.html renamed to pontoon/base/templates/allauth/layouts/base.html pontoon/base/templates/django/base.html renamed to pontoon/base/templates/allauth/layouts/base.html

@@ -1,4 +1,4 @@
-{% extends 'django/base.html' %}
+{% extends "allauth/layouts/base.html" %}
 
 {% block content %}
 <h1 id="title">Pontoon</h1>

pontoon/base/templates/django/account/base.html

@@ -1,4 +1,4 @@
-{% extends "socialaccount/base.html" %}
+{% extends "allauth/layouts/base.html" %}
 
 {% load socialaccount %}
 

pontoon/base/templates/django/socialaccount/base.html

@@ -8,7 +8,7 @@
 
 import markupsafe
 
-from allauth.socialaccount import providers
+from allauth.socialaccount.adapter import get_adapter
 from allauth.utils import get_request_param
 from bleach.linkifier import Linker
 from django_jinja import library
@@ -210,7 +210,7 @@ def provider_login_url(request, provider_id=settings.AUTHENTICATION_METHOD, **qu
     This function adapts the django-allauth templatetags that don't support jinja2.
     @TODO: land support for the jinja2 tags in the django-allauth.
     """
-    provider = providers.registry.by_id(provider_id)
+    provider = get_adapter().get_provider(request, provider_id)
 
     auth_params = query.get("auth_params", None)
     process = query.get("process", None)
@@ -233,9 +233,9 @@ def provider_login_url(request, provider_id=settings.AUTHENTICATION_METHOD, **qu
 @library.global_function
 def providers_media_js(request):
     """A port of django tag into jinja2"""
-    return markupsafe.Markup(
-        "\n".join([p.media_js(request) for p in providers.registry.get_list()])
-    )
+    providers_list = get_adapter(request).list_providers(request)
+    ret = "\n".join(p.media_js(request) for p in providers_list)
+    return markupsafe.Markup(ret)
 
 
 @library.filter

pontoon/base/templates/registration/login.html

@@ -10,8 +10,8 @@ $(function () {
         csrfmiddlewaretoken: $('body').data('csrf'),
         value: self.is('.enable'),
       },
-      success: function (data) {
-        window.location.href = data.next;
+      success() {
+        window.location.href = '/';
       },
       error: function (request) {
         if (request.responseText === 'error') {

…/socialaccount/authentication_error.html …/socialaccount/authentication_error.htmlpontoon/base/templates/django/socialaccount/authentication_error.html renamed to pontoon/base/templates/socialaccount/authentication_error.html pontoon/base/templates/django/socialaccount/authentication_error.html renamed to pontoon/base/templates/socialaccount/authentication_error.html

@@ -19,7 +19,6 @@ def test_dismiss_email_consent(member):
     assert profile.email_communications_enabled is False
     assert profile.email_consent_dismissed_at is not None
     assert response.status_code == 200
-    assert response.json()["next"] == "/"
 
     params = {
         "value": "true",
@@ -29,4 +28,3 @@ def test_dismiss_email_consent(member):
     assert profile.email_communications_enabled is True
     assert profile.email_consent_dismissed_at is not None
     assert response.status_code == 200
-    assert response.json()["next"] == "/"